hdfirmware / firmware / App

Controllers

Clone Tools

Stats

Commits this week: 0
Total Committers: 16
Last Commit: 19 Nov 25
Commits this week: 0

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 04 Nov 2022
1:02 pm

Dara Navaei

12:09 pm

wbracken  (deleted user)

Thought we had decided that the standard would be to add the F?

Thought we had decided that the standard would be to add the F?

11:57 am

Sean Nash

Ok to calculate here (this is called before UI needs it I assume). I was expecting this to be pre-calculated on exit of treatment params mode or on entry to pre-treatment mode when we have what we ...

Ok to calculate here (this is called before UI needs it I assume). I was expecting this to be pre-calculated on exit of treatment params mode or on entry to pre-treatment mode when we have what we need to do the calculation.

11:43 am

Sean Nash

I would think this alarm would be triggered generally by driver because current mode said we shouldn't see syringe yet. Can we just set stopPump to TRUE here and nothing else?

I would think this alarm would be triggered generally by driver because current mode said we shouldn't see syringe yet.
Can we just set stopPump to TRUE here and nothing else?

11:37 am

Sean Nash

No harm in adding "F" suffix, but not necessary for assignment to a float variable - assignment statement is inherently typed. We need the "F" suffix where typing is not assured (e.g. #define where...

No harm in adding "F" suffix, but not necessary for assignment to a float variable - assignment statement is inherently typed. We need the "F" suffix where typing is not assured (e.g. #define where we don't know how it will be used).

11:33 am

Sean Nash

We probably don't need this flag at this point, but should we reset this flag on exit of retract (after treatment done)?

We probably don't need this flag at this point, but should we reset this flag on exit of retract (after treatment done)?

11:26 am

Sean Nash

I think this should probably be reset elsewhere. Init function and on exit of retract maybe?

I think this should probably be reset elsewhere. Init function and on exit of retract maybe?

11:16 am

Sean Nash

Probably could just be if ( TRUE == isSyringePumpPreLoaded() ).

Probably could just be if ( TRUE == isSyringePumpPreLoaded() ).

Thursday 03 Nov 2022
1:42 pm

wbracken  (deleted user)

0.0F

0.0F

1:42 pm

wbracken  (deleted user)

0.0F

0.0F

1:41 pm

wbracken  (deleted user)

0.0F

0.0F

12:42 pm

Dara Navaei

HD-DEN-14170_SW Dev Sprint 82 Darren
HD-DEN-14170_SW Dev Sprint 82 Darren
10:23 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

10:23 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

10:20 am

wbracken  (deleted user)

Update function header.

Update function header.

10:17 am

wbracken  (deleted user)

Remove blank line

Remove blank line

9:52 am

Dara Navaei

Done.

Done.

9:51 am

Dara Navaei

This function should stay commented. I have a TODO task for this.

This function should stay commented. I have a TODO task for this.

9:50 am

Dara Navaei

Done.

Done.

9:49 am

Dara Navaei

No these should be TRUE.

No these should be TRUE.

9:04 am

Sean Nash

Add a space between "return" and "(".

Add a space between "return" and "(".

Friday 28 Oct 2022
3:13 pm

Michael Garthwaite

3:13 pm

Sean Nash

hasReg1Failed is initialized to FALSE. Should we be setting to TRUE in these ifs? Or should the initialize be setting to TRUE?

hasReg1Failed is initialized to FALSE. Should we be setting to TRUE in these ifs? Or should the initialize be setting to TRUE?

3:09 pm

Sean Nash

This revision also applies to UI.

This revision also applies to UI.

3:07 pm

Sean Nash

3:07 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

3:07 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

3:07 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

3:07 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

3:07 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.