Activity dgfirmware/firmware/App/Services

10:32 am

Sean Nash

More
99b0c8f
changed 14 files
DEN-11114-dg_dev-dg-dialysate-temperature-control-2-of-2, DEN-11750-dg_dev-dg-dialysate-temperature-control-tune-up and 214 more DG-DEN-7605-1 DG-DEN-8030-1 DG-DEN-8103-1 DG-DEN-7802-1

DEN-7605: Local merge w/ staging.

- -1
- +2
./AlarmMgmt.h
- History
- Source
- Diff
./AlarmMgmt.h
- -252
- +254
./Reservoirs.c
- History
- Source
- Diff
./Reservoirs.c
- -16
- +15
./Reservoirs.h
- History
- Source
- Diff
./Reservoirs.h
- -0
- +4
./SystemComm.c
- History
- Source
- Diff
./SystemComm.c
- -0
- +63
./SystemCommMessages.c
- History
- Source
- Diff
./SystemCommMessages.c
- -0
- +7
./SystemCommMessages.h
- History
- Source
- Diff
./SystemCommMessages.h
… 8 more files in changeset.

9:34 am

Sean Nash

More
Re: DG-DEN-7802-1
commented on review

I think we should not stop RO pump or heater. Just go to drain like production state (but no mixing).

9:31 am

Sean Nash

More
Re: DG-DEN-7802-1
commented on review

Call checkDialysateConductivity() once at top of function. Then use return value for the two if conditions. Also, this second if should be an else if to ensure you don't do both.

Call checkDialysateConductivity() once at top of function. Then use return value for the two if conditions.
Also, this second if should be an else if to ensure you don't do both.

11:41 pm

Quang Nguyen <qnguyen@diality.com>

More
bb1f2d0
changed 1 file
DEN-11114-dg_dev-dg-dialysate-temperature-control-2-of-2, DEN-11750-dg_dev-dg-dialysate-temperature-control-tune-up and 212 more DG-DEN-8103-1

DEN-8103: Transition to fault mode if alarm is a DG fault alarm

- -2
- +2
./AlarmMgmt.c
- History
- Source
- Diff
./AlarmMgmt.c

1:57 pm

Sean Nash

More
7df1fae
changed 8 files
develop_1-24

Fixed merge conflicts.

- -9
- +9
./FPGA.c
- History
- Source
- Diff
./FPGA.c
- -2
- +2
./FPGA.h
- History
- Source
- Diff
./FPGA.h
- -2
- +2
./SystemCommMessages.c
- History
- Source
- Diff
./SystemCommMessages.c
… 5 more files in changeset.

1:50 pm

Sean Nash

More
5f86e35
changed 7 files
DEN-11114-dg_dev-dg-dialysate-temperature-control-2-of-2, DEN-11750-dg_dev-dg-dialysate-temperature-control-tune-up and 214 more DG-DEN-7605-1 DG-DEN-8103-1

DEN-7605: Minor fixes.

- -9
- +9
./FPGA.c
- History
- Source
- Diff
./FPGA.c
- -2
- +2
./FPGA.h
- History
- Source
- Diff
./FPGA.h
- -2
- +2
./SystemCommMessages.c
- History
- Source
- Diff
./SystemCommMessages.c
… 4 more files in changeset.

6:13 pm

Dara Navaei

More
3f7d30b
changed 10 files
DEN-11114-dg_dev-dg-dialysate-temperature-control-2-of-2, DEN-11750-dg_dev-dg-dialysate-temperature-control-tune-up and 213 more DG-DEN-8030-1

DEN-8030 added UI states to heat disinfect. Changed the heaters control.

- -2
- +2
./SystemCommMessages.c
- History
- Source
- Diff
./SystemCommMessages.c
… 9 more files in changeset.

2:13 pm

Sean Nash

More
Re: DG-DEN-8030-1
commented on review

Change order of comparison.

2:11 pm

Sean Nash

More
Re: DG-DEN-8030-1
commented on review

Blank line after declaration.

2:08 pm

Sean Nash

More
Re: DG-DEN-8030-1
commented on review

I think we should add the new THd temperature sensor now. I know it's currently using TRo connection (and so we don't have a TRo), but we should handle that with build switch (THd reads from TRo ch...

I think we should add the new THd temperature sensor now. I know it's currently using TRo connection (and so we don't have a TRo), but we should handle that with build switch (THd reads from TRo channel and TRo can just be a copy of TDi for now if THD_USING_TRO_CONNECTOR build switch enabled, actual future code where both sensors exist on their own connectors if build switch is disabled).

12:58 pm

Quang Nguyen <qnguyen@diality.com>

More
acf0804
changed 2 files
DEN-11114-dg_dev-dg-dialysate-temperature-control-2-of-2, DEN-11750-dg_dev-dg-dialysate-temperature-control-tune-up and 212 more DG-DEN-8103-1

DEN-8103: Added alarm table size check

- -0
- +3
./AlarmMgmt.c
- History
- Source
- Diff
./AlarmMgmt.c
… 1 more file in changeset.

12:42 pm

Sean Nash

More
Re: DG-DEN-8030-1
commented on review

Is this needed. Should we just let Standby mode set actuators as it wants on entry?

12:40 pm

Sean Nash

More
Re: DG-DEN-8030-1
commented on review

I think we should at least say that the mode variables are reset to re-start the mode or something.

12:37 pm

Sean Nash

More
Re: DG-DEN-8030-1
commented on review

Why are some values aligned at left and others at right? I think all values should align at left.

12:32 pm

Sean Nash

More
Re: DG-DEN-8030-1
commented on review

I'm assuming HET was enabled to get access to a GPIO pin? If so, do we need these notifications? Is there a place in HalCOGen to disable HET interrupts?

12:28 pm