dd-firmware / firmware

source

Clone Tools

Stats

Commits this week: 2
Total Committers: 9
Last Commit: 18 Jun
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
Thursday 18 Jun
10:10 am

Vendor - TEL - Sameer Poyil

LDT-4612 integrating fix for concentrate pump not running into develop

Options
    • -0
    • +1
    ./sys_main.c
  2. … 6 more files in changeset.
10:06 am

Vendor - TEL - Sameer Poyil

LDT-4409 fix for concentrate pump not running

Options
    • -0
    • +1
    ./sys_main.c
  2. … 2 more files in changeset.
4:02 am

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-4612_Dialysate Composition (Dry Bicarbonate) - FW Iteration - 4/1:
LEAHI-DD-FIRMWARE-LDT-4612_Dialysate Composition (Dry Bicarbonate) - FW Iteration - 4/1:
Thursday 11 Jun
3:44 pm

Vendor - TEL - Arpita Srivastava

LDT-3735: Moved the file NVDriver, NVJobQ and NVMgmt to FwCommon

Options
    • -2
    • +2
    ./sys_main.c
  2. … 24 more files in changeset.
Tuesday 09 Jun
5:03 pm

Vendor - TEL - Arpita Srivastava

LDT-3735: Added function wrappers for public functions

Options
    • -0
    • +2
    ./sys_main.c
  2. … 18 more files in changeset.
Wednesday 03 Jun
3:05 pm

Steve Jarpe <sjarpe@diality.com>

LDT-4002: Initial version of Switches module for HDF

Options
    • -275
    • +0
    ./sys_main.c
  2. … 5 more files in changeset.
Wednesday 27 May
4:24 pm

Dara Navaei

4:22 pm

Dara Navaei

Resolved the conflicts from staging

Options
    • -3
    • +0
    ./sys_main.c
4:20 pm

Dara Navaei

Merged staging and resolved conflicts

Options
    • -0
    • +5
    ./sys_main.c
  2. … 11 more files in changeset.
3:38 pm

Vendor - TEL - Jashwant Gantyada and Sean Nash

2:46 pm

Sean Nash

On DD f/w side, this should just be a simple pump controller. States should be Off and On (control to commanded Qsub). There is no flow sensor for feedback control, so assuming this is open loop si...

On DD f/w side, this should just be a simple pump controller. States should be Off and On (control to commanded Qsub). There is no flow sensor for feedback control, so assuming this is open loop similar to UF pump.
GenDialysate mode will have an HDF sub-state that signals this controller from Off to On and would also call separate controller for heater and separate (Switches) monitor for door sensors.

With all of that in mind, I think this unit should have a simpler name like SubPump.

2:39 pm

Sean Nash

I think there should be some state monitoring for potential alarm(s) in here too. Similar to BP door on TD f/w side where the current mode/state can signal this unit that it expects specific switch...

I think there should be some state monitoring for potential alarm(s) in here too. Similar to BP door on TD f/w side where the current mode/state can signal this unit that it expects specific switch state(s) (e.g. while in HDF state, we would expect the cap to be open and stowed and we would want to alarm if that was not the case).

2:36 pm

Sean Nash

Remove extra blank line.

Remove extra blank line.

2:33 pm

Dara Navaei

LDT-3272 addressed the code review comments

Options
    • -10
    • +10
    ./can.c
  2. … 3 more files in changeset.
2:31 pm

Sean Nash

Where is call to HDF controller?

Where is call to HDF controller?

2:31 pm

Sean Nash

Monitors should come before mode mgmt above.

Monitors should come before mode mgmt above.

2:30 pm

Sean Nash

What about HemoDiaFiltration.h?

What about HemoDiaFiltration.h?

2:29 pm

Sean Nash

What about initHDF?

What about initHDF?

2:28 pm

Sean Nash

Where is HemoDiaFiltration.h?

Where is HemoDiaFiltration.h?

2:24 pm

Sean Nash

Why does this file name have pound signs pre/post?

Why does this file name have pound signs pre/post?

2:14 pm

Sean Nash

This macro is already defined in Common.h.

This macro is already defined in Common.h.

2:14 pm

Sean Nash

Align all "=" or none of them.

Align all "=" or none of them.

2:13 pm

Sean Nash

DD HDF control data published should just be state and set rate. TD HDF data would have volume, etc...

DD HDF control data published should just be state and set rate. TD HDF data would have volume, etc...

2:11 pm

Sean Nash

This looks like pump control function. I think TD f/w should be doing this when setting Qsub before sending to DD f/w. Again, I think DD f/w should just be controlling pump to the Qsub rate given b...

This looks like pump control function. I think TD f/w should be doing this when setting Qsub before sending to DD f/w. Again, I think DD f/w should just be controlling pump to the Qsub rate given by TD f/w.
HDF volume should be calculated by TD f/w by integrating commanded Qsub over time.

2:08 pm

Sean Nash

Remove blank line.

Remove blank line.

2:08 pm

Sean Nash

Refactor so that only one return statement at end of function. This is in our coding standards.

Refactor so that only one return statement at end of function. This is in our coding standards.

2:08 pm

Dara Navaei

This file has been updated by HALCoGen automatically.

This file has been updated by HALCoGen automatically.

2:07 pm

Dara Navaei

This file has been updated by HALCoGen automatically.

This file has been updated by HALCoGen automatically.

2:07 pm

Sean Nash

I think TD should be handling all of these rules before setting Qsub and sending to DD.

I think TD should be handling all of these rules before setting Qsub and sending to DD.

2:06 pm

Dara Navaei

This file has been updated by HALCoGen automatically.

This file has been updated by HALCoGen automatically.