hdfirmware / firmware

App

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 12 May 2023
1:48 pm

wbracken  (deleted user)

Removed

Removed

1:47 pm

wbracken  (deleted user)

There are enums for the valves. However, the enum value does not correspond to their position in the valve status returned from the FPGA. For example, VDI is enumerated to be the first value in the...

There are enums for the valves. However, the enum value does not correspond to their position in the valve status returned from the FPGA. For example, VDI is enumerated to be the first value in the enum but is the second 2nd group of 3 bits (0 relative) returned by the FPGA.

1:39 pm

wbracken  (deleted user)

This code is identical to the DG implementation. Initially they were only included in debug mode. Seemed like it made sense to move to function as they are only used in the function.

This code is identical to the DG implementation. Initially they were only included in debug mode. Seemed like it made sense to move to function as they are only used in the function.

1:33 pm

wbracken  (deleted user)

This code is identical to the DG implementation. Initially they were only included in debug mode. Seemed like it made sense to move to function as they are only used in the function.

This code is identical to the DG implementation. Initially they were only included in debug mode. Seemed like it made sense to move to function as they are only used in the function.

1:31 pm

wbracken  (deleted user)

Corrected

Corrected

1:21 pm

wbracken  (deleted user)

Removed

Removed

1:20 pm

wbracken  (deleted user)

Added space. I modified slightly, Since this is called outside of the interrupt where the sci2FEOEError is being set I believe it should get the value and only reset it if was TRUE.

Added space. I modified slightly, Since this is called outside of the interrupt where the sci2FEOEError is being set I believe it should get the value and only reset it if was TRUE.

1:16 pm

wbracken  (deleted user)

They were used when FE OE generated software fault. Removed.

They were used when FE OE generated software fault. Removed.

1:15 pm

wbracken  (deleted user)

Removed.

Removed.

1:13 pm

wbracken  (deleted user)

Cut and paste error from DG. Corrected.

Cut and paste error from DG. Corrected.

1:07 pm

wbracken  (deleted user)

Removed. Replaced fpgaReadByteSize with sizeof( FPGA_SENSORS_T ) in the two statements that it was being used.

Removed. Replaced fpgaReadByteSize with sizeof( FPGA_SENSORS_T ) in the two statements that it was being used.

1:07 pm

Sean Nash

I don't see these two functions in the .c file. Are they not yet implemented?

I don't see these two functions in the .c file. Are they not yet implemented?

1:06 pm

Sean Nash

Why 2 of these?

Why 2 of these?

1:02 pm

Sean Nash

What does this do? Why 64 (not 32)?

What does this do? Why 64 (not 32)?

1:01 pm

wbracken  (deleted user)

Done

Done

12:53 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:52 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:49 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

11:44 am

wbracken  (deleted user)

Nothing else needs to be done.

Nothing else needs to be done.

11:42 am

Dara Navaei

HD-DEN-15196_DIAL-251
HD-DEN-15196_DIAL-251
11:38 am

Sean Nash

With that approach, we are likely to have pump/heater off for some time which will allow the dialysate to cool and perhaps cause a temperature alarm when we do start treatment. I think we should ha...

With that approach, we are likely to have pump/heater off for some time which will allow the dialysate to cool and perhaps cause a temperature alarm when we do start treatment.
I think we should have the pump/heater on when door is closed in patient connect states.

11:33 am

Sean Nash

Next state only requires door closed after confirm (when state is done). In meantime, door can be opened while pump is on and no alarm will occur and pump stays on I would think.

Next state only requires door closed after confirm (when state is done). In meantime, door can be opened while pump is on and no alarm will occur and pump stays on I would think.

11:11 am

Sean Nash

Remove blank line.

Remove blank line.

11:09 am

Sean Nash

Do we need to check for these in out exec?

Do we need to check for these in out exec?

11:07 am

Sean Nash

Are these used anywhere?

Are these used anywhere?

11:06 am

Sean Nash

Add space between ( and TRUE. Or just remove condition and always set flag to FALSE here.

Add space between ( and TRUE. Or just remove condition and always set flag to FALSE here.

11:02 am

Sean Nash

Remove blank line.

Remove blank line.

11:01 am

Sean Nash

Use enums instead of literals.

Use enums instead of literals.

11:01 am

Sean Nash

Don't we have an enum for these values?

Don't we have an enum for these values?

10:58 am

Sean Nash

Is there more to be done here (TODO)?

Is there more to be done here (TODO)?