dd-firmware / firmware / App / Controllers

ConcentratePumps.h

Clone Tools

Stats

Commits this week: 2
Total Committers: 2
Last Commit: 11 Dec
Commits this week: 2

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 10 Dec
4:32 pm

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -2
    • +2
    ./ConcentratePumps.h
  2. … 121 more files in changeset.
Monday 08 Dec
1:02 pm

Sean Nash

Move up with other controllers.

Move up with other controllers.

Friday 05 Dec
1:26 pm

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -2
    • +2
    ./ConcentratePumps.h
  2. … 121 more files in changeset.
Thursday 16 Oct
9:52 am

Sean Nash

I only see these configurations being set here in initialize function. Is that all we need? Is this configuration what we want for these valves in all states? I would recommend we combine these 3 ...

I only see these configurations being set here in initialize function. Is that all we need? Is this configuration what we want for these valves in all states?

I would recommend we combine these 3 fpga functions into one single function that takes all 3 params and updates all 3 registers.

9:35 am

Sean Nash

I think these should be initialized to zero in case there are unused bits that won't be set below.

I think these should be initialized to zero in case there are unused bits that won't be set below.

9:30 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:26 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

Tuesday 07 Oct
2:15 pm

jpaguio  (deleted user)

2:15 pm

jpaguio  (deleted user)

LDT-2198 was merged into this branch after being merged into staging_alpha. Branch will be killed and relevant code changes were reviewed appropriately.

LDT-2198 was merged into this branch after being merged into staging_alpha. Branch will be killed and relevant code changes were reviewed appropriately.

9:46 am

Sean Nash

Monday 06 Oct
11:01 am

Sean Nash and Vinayakam Mani

11:00 am

Raghu Kallala , Michael Garthwaite and Vinayakam Mani

Friday 03 Oct
2:45 pm

Vinayakam Mani

Done.

Done.

Wednesday 01 Oct
10:47 am

Vinayakam Mani

Tuesday 30 Sep
1:33 pm

Vinayakam Mani

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

11:45 am

jpaguio  (deleted user)

Done

Done

11:45 am

jpaguio  (deleted user)

Done

Done

11:45 am

jpaguio  (deleted user)

Done

Done

11:44 am

jpaguio  (deleted user)

Done

Done

11:44 am

jpaguio  (deleted user)

Done

Done

9:45 am

Sean Nash

Appears to be dialysate to dialyzer flow rate.

Appears to be dialysate to dialyzer flow rate.

9:44 am

Sean Nash

Looking at the implementation of this function, it appears that this function is setting dialysateFlowRate to target rate only when delivering dialysate to the dialyzer and to zero in all other cas...

Looking at the implementation of this function, it appears that this function is setting dialysateFlowRate to target rate only when delivering dialysate to the dialyzer and to zero in all other cases. Consider changing variable name to something like dialysateToDialyzerFlowRate and updating function description in header to make this more clear.

9:39 am

Sean Nash

0.0F.

0.0F.

9:21 am

Vinayakam Mani

Check the current Gen dialysate sub state.

Check the current Gen dialysate sub state.

9:20 am

Vinayakam Mani

Please use 0.0F (since its float variable).

Please use 0.0F (since its float variable).

Friday 26 Sep
10:04 am

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-1196_DD: Request for a variable for target Qd rate
LEAHI-DD-FIRMWARE-LDT-1196_DD: Request for a variable for target Qd rate
Thursday 25 Sep
8:48 am

Sean Nash

Why the extra leading zeroes? If you want to ensure the literal is treated like a U32, you can do 0x00000001 or simply 1U, but I suspect the compiler will default to 32 bits anyway.

Why the extra leading zeroes? If you want to ensure the literal is treated like a U32, you can do 0x00000001 or simply 1U, but I suspect the compiler will default to 32 bits anyway.

Wednesday 24 Sep
9:10 am

Sean Nash

I believe baro is being moved to the TD firmware. DD will have to get baro readings from TD broadcasts.

I believe baro is being moved to the TD firmware. DD will have to get baro readings from TD broadcasts.

Tuesday 23 Sep
10:23 am

jpaguio  (deleted user)

Do you want to add a space for the baro pressure sensor from which we receive from the TD?

Do you want to add a space for the baro pressure sensor from which we receive from the TD?

Monday 22 Sep
10:11 am

Dara Navaei and Sean Nash