leahi-fwcommon

Clone Tools

Stats

Commits this week: 6
Total Committers: 12
Last Commit: 02 Mar
Commits this week: 6

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
Monday 02 Mar
1:13 pm

Michael Garthwaite

Merge branch 'LDT-3514-dialin-config' into develop

Options
    • -2
    • +3
    /TestSupport.h
1:12 pm

Michael Garthwaite

LDT-3514: implemented RO current test config

Options
    • -0
    • +1
    /TestSupport.h
1:03 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3514_Dialin config request to disable IOFP current monitoring
LEAHI-DD-FIRMWARE-LDT-3514_Dialin config request to disable IOFP current monitoring
10:18 am

Sean Nash

Actually, I think we should have the local variable to copy the payload into, then we should range check it (and potentially reject it if out of range), and if it is in range, only then should we a...

Actually, I think we should have the local variable to copy the payload into, then we should range check it (and potentially reject it if out of range), and if it is in range, only then should we assign new rate to your static variable.

10:15 am

Sean Nash

I think this can just be an else since resume is the only remaining possibility.

I think this can just be an else since resume is the only remaining possibility.

10:10 am

Sean Nash

Should we set resume state here? I know it's initialized to ramp state already, but in case we come back to ramp it may be safer to set it here.

Should we set resume state here? I know it's initialized to ramp state already, but in case we come back to ramp it may be safer to set it here.

9:59 am

Sean Nash

You moved it, but didn't change anything. The comment says it's a timer counter but the variable name suggests it is actually a flow rate. I want you to fix the comment.

You moved it, but didn't change anything. The comment says it's a timer counter but the variable name suggests it is actually a flow rate. I want you to fix the comment.

9:56 am

Sean Nash

I think system messages (non Dialin (test) functions should come first in this table so that they are found faster.

I think system messages (non Dialin (test) functions should come first in this table so that they are found faster.

Friday 27 Feb
12:01 pm

Vendor - TEL - Varshini Nagabooshanam

Added

Added

12:01 pm

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

12:00 pm

Vendor - TEL - Varshini Nagabooshanam

Removed and replaced

Removed and replaced

12:00 pm

Vendor - TEL - Varshini Nagabooshanam

Added

Added

12:00 pm

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

12:00 pm

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

12:00 pm

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

12:00 pm

Vendor - TEL - Varshini Nagabooshanam

Assigned new variable

Assigned new variable

12:00 pm

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

11:59 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

11:59 am

Vendor - TEL - Varshini Nagabooshanam

Added

Added

11:59 am

Vendor - TEL - Varshini Nagabooshanam

Added

Added

11:59 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

11:59 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

11:59 am

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

11:59 am

Vendor - TEL - Varshini Nagabooshanam

Added

Added

11:59 am

Vendor - TEL - Varshini Nagabooshanam

Added

Added

11:58 am

Vendor - TEL - Varshini Nagabooshanam

Added back

Added back

11:58 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

Thursday 26 Feb
11:24 pm

Sean Nash

Response payload is not U08. I believe UI will be expecting a payload of type UI_RESPONSE_PAYLOAD_T (defined in Messaging.h). Verify with Nico that this is the payload he is expecting in response. ...

Response payload is not U08. I believe UI will be expecting a payload of type UI_RESPONSE_PAYLOAD_T (defined in Messaging.h). Verify with Nico that this is the payload he is expecting in response. See signalPauseResumeUF() function for example of how to respond to a UI request.

11:24 pm

Sean Nash

Same comment as other message handling function. Use correct payload type. See signalPauseResumeUF() function for example of how to respond to a UI request.

Same comment as other message handling function. Use correct payload type. See signalPauseResumeUF() function for example of how to respond to a UI request.

11:19 pm

Sean Nash

Why do we need this local variable when we have the static requestedBloodFlowRate_mL_min variable?

Why do we need this local variable when we have the static requestedBloodFlowRate_mL_min variable?