hdfirmware / firmware / App / Modes

Dialysis.c

Clone Tools

Stats

Commits this week: 0
Total Committers: 12
Last Commit: 19 Nov
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
Wednesday 12 Oct 2022
9:16 am

Sean Nash

This probably warrants a comment.

This probably warrants a comment.

9:12 am

Sean Nash

Is this a TODO? Can we just do it?

Is this a TODO? Can we just do it?

9:11 am

Sean Nash

Why are we checking non-zero here? Seems like we will get stuck here if years really is set to zero.

Why are we checking non-zero here? Seems like we will get stuck here if years really is set to zero.

9:09 am

Michael Garthwaite

This whole block of code ( lines 127 - 178 ) was removed in when staging was merged in and not reflected on crucible.

This whole block of code ( lines 127 - 178 ) was removed in when staging was merged in and not reflected on crucible.

9:01 am

Michael Garthwaite

This whole block of code ( lines 248 - 304 ) was removed in when staging was merged in and not reflected on crucible.

This whole block of code ( lines 248 - 304 ) was removed in when staging was merged in and not reflected on crucible.

8:40 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

8:38 am

Sean Nash

Remove last param.

Remove last param.

8:37 am

Sean Nash

Remove last param.

Remove last param.

8:37 am

Sean Nash

Remove last param.

Remove last param.

8:30 am

Sean Nash

Remove param.

Remove param.

8:29 am

Sean Nash

Add space before last ")"

Add space before last ")"

Thursday 06 Oct 2022
3:42 pm

wbracken  (deleted user)

Does this need to handle lowercase hex characters and invalid characters?

Does this need to handle lowercase hex characters and invalid characters?

Tuesday 04 Oct 2022
5:53 pm

Sean Nash

Why not checking payload length?

Why not checking payload length?

5:52 pm

Sean Nash

Keep blank line.

Keep blank line.

5:48 pm

Sean Nash

These FPGA registers are obsolete - h/w is no longer in our design. Need to continue to maintain register space since registers after these have not moved. You can think of these obsolete registers...

These FPGA registers are obsolete - h/w is no longer in our design. Need to continue to maintain register space since registers after these have not moved. You can think of these obsolete registers as "reserved" for future purpose.
Dara, I do think we could rename these register fields to something like Reserved1, Reserved2, ...

5:43 pm

Sean Nash

Monitor execs should be before operation modes exec.

Monitor execs should be before operation modes exec.

5:37 pm

Sean Nash

Why was blank line removed?

Why was blank line removed?

5:29 pm

Sean Nash

Align "=".

Align "=".

5:27 pm

Sean Nash

Add another blank line here.

Add another blank line here.

5:22 pm

Sean Nash

Idea is to re-purpose these (instead of adding new alarms to bottom of enum). We prefer not to collapse the table when an alarm ID goes obsolete so other alarm IDs do not change.

Idea is to re-purpose these (instead of adding new alarms to bottom of enum). We prefer not to collapse the table when an alarm ID goes obsolete so other alarm IDs do not change.

11:07 am

wbracken  (deleted user)

What is the significance of (OBSOLETE)? Applies to several blocks.

What is the significance of (OBSOLETE)? Applies to several blocks.

Monday 03 Oct 2022
10:48 am

wbracken  (deleted user)

fpgaPersistentAlarmGroup is an input.

fpgaPersistentAlarmGroup is an input.

10:44 am

wbracken  (deleted user)

Are all available ALARM_IDs being removed?

Are all available ALARM_IDs being removed?

Friday 30 Sep 2022
9:00 am

Michael Garthwaite

Do we want this checked into staging?

Do we want this checked into staging?

Thursday 29 Sep 2022
5:17 pm

Sean Nash

These are defined in the Code Composer project for DG and HD firmware respectively. You can only have one defined and you must have one defined.

These are defined in the Code Composer project for DG and HD firmware respectively. You can only have one defined and you must have one defined.

5:06 pm

Sean Nash

What is the purpose of this "no event" event?

What is the purpose of this "no event" event?

5:06 pm

Sean Nash

What is the purpose of this "no event" event?

What is the purpose of this "no event" event?

5:04 pm

Sean Nash

No stop property for a high priority alarm? PRS 754 says dialysate temperature alarms are low priority. Temp too low = rank 902. Temp too high = rank 901.

No stop property for a high priority alarm?
PRS 754 says dialysate temperature alarms are low priority. Temp too low = rank 902. Temp too high = rank 901.

4:54 pm

Sean Nash

Why no stop?

Why no stop?

4:53 pm

Sean Nash

Why no stop?

Why no stop?