leahi-common

Clone Tools

Stats

Commits this week: 18
Total Committers: 18
Last Commit: 28 Jan
Commits this week: 18

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 08 May
1:21 pm

Sean Nash

Again, I don't like these oriented this way. Please change to #ifndef DISABLE_ALARM_4

Again, I don't like these oriented this way. Please change to #ifndef DISABLE_ALARM_4

1:19 pm

Sean Nash

If BC switching period is Zero, I would expect the Qd from TD to also be zero, right? But what if it's not? What if Qd from TD is 600 and switching period is zero? You are going to run the pump pre...

If BC switching period is Zero, I would expect the Qd from TD to also be zero, right? But what if it's not? What if Qd from TD is 600 and switching period is zero? You are going to run the pump pretty fast with no BC switching in that case.

1:10 pm

Sean Nash

We shouldn't require something to be defined to do the right thing. We should require something to be defined to do the wrong thing (i.e. this should say #ifndef _BAD_MSG_CRC_ALARM_DISABLED_) And t...

We shouldn't require something to be defined to do the right thing. We should require something to be defined to do the wrong thing (i.e. this should say #ifndef _BAD_MSG_CRC_ALARM_DISABLED_)
And then, we don't need a TODO. These #defines are only possible in DEBUG builds. A RELEASE build will never allow thing build flags to disable things.

1:00 pm

Sean Nash and Vendor - TEL - Varshini Nagabooshanam

Thursday 07 May
11:12 am

Sean Nash

Remove start state.

Remove start state.

10:30 am

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

10:29 am

Vendor - TEL - Varshini Nagabooshanam

updated

updated

10:29 am

Vendor - TEL - Varshini Nagabooshanam

Yes, removed

Yes, removed

10:29 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

10:28 am

Vendor - TEL - Varshini Nagabooshanam

yes, removed

yes, removed

10:28 am

Vendor - TEL - Varshini Nagabooshanam

Changed

Changed

10:28 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

10:28 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

10:27 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

10:27 am

Vendor - TEL - Varshini Nagabooshanam

Added

Added

10:27 am

Vendor - TEL - Varshini Nagabooshanam

Added

Added

10:27 am

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

10:27 am

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

10:27 am

Vendor - TEL - Varshini Nagabooshanam

Added

Added

10:26 am

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

10:25 am

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

9:31 am

Vendor - TEL - Jashwant Gantyada and Sean Nash

9:31 am

Vendor - TEL - Jashwant Gantyada

Wednesday 06 May
5:55 pm

Sean Nash

Same comment as duration confirmation. compare uf rate too.

Same comment as duration confirmation. compare uf rate too.

5:44 pm

Sean Nash

Also check uf rate from confirmation message (request) is still the same as validated uf rate. WARNING - uf rate will be a float and we shouldn't use == operator on floats.

Also check uf rate from confirmation message (request) is still the same as validated uf rate.
WARNING - uf rate will be a float and we shouldn't use == operator on floats.

5:38 pm

Sean Nash

I think validated variable here should be L/Hr and assigned to response.ufRate so that when we get confirmation, we can compare rates with same units.

I think validated variable here should be L/Hr and assigned to response.ufRate so that when we get confirmation, we can compare rates with same units.

5:38 pm

Sean Nash

I think validated variable here should be L/Hr and assigned to response.ufRate so that when we get confirmation, we can compare rates with same units.

I think validated variable here should be L/Hr and assigned to response.ufRate so that when we get confirmation, we can compare rates with same units.

11:43 am

Sean Nash

Treatment mode will also need one of these (later). Should we add PreTx to the function name to differentiate?

Treatment mode will also need one of these (later). Should we add PreTx to the function name to differentiate?

10:02 am

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3910_Request: add TD config flag so that alarms originating from other subsystems don't put the TD into fault mode
LEAHI-DD-FIRMWARE-LDT-3910_Request: add TD config flag so that alarms originating from other subsystems don't put the TD into fault mode
Tuesday 05 May
6:54 pm

Sean Nash

I don't see you changing the GoalL. Recommend you use local presUFVolumeMl instead.

I don't see you changing the GoalL. Recommend you use local presUFVolumeMl instead.