Activity leahi-dialin/leahi_dialin/dd/modules/rinse

Vinayakam Mani What is the advantage of doing that? If I go with Beta_1_9_HW, I will need to rename the flag when we start using Beta 2+. And rename it in all the scripts in different repository wh...

Vinayakam Mani What is the advantage of doing that?
If I go with Beta_1_9_HW, I will need to rename the flag when we start using Beta 2+. And rename it in all the scripts in different repository when starting with 2.0.
While I use a more generic flag, I won't need to do that. So just using a better name I am sparing myself hours of work in the future.

5:32 pm

Zoltan Miskolci

More
0bdafdb
changed 23 files
LDT-2004-dialysate-composition-dd, LDT-2004-dry-bicart---dd and 16 more LEAHI-DIALIN-LDT-3214-1

LDT-3214 - Refactored DD module's override functions to use the override template functions - Removed constants file from DD module to use the one from common

- -34
- +19
./rinse_pump.py
- History
- Source
- Diff
./rinse_pump.py
… 22 more files in changeset.

11:47 am

Vinayakam Mani

More
Re: LEAHI-DIALIN-LDT-2998-1
commented on review

You may use the same test configs from DD here, to set the relevant test configs from FP Dialin.

11:35 am

Vinayakam Mani

More
Re: LEAHI-DIALIN-LDT-2998-1
commented on review

POS_FIELD_3??

11:34 am

Vinayakam Mani

More
Re: LEAHI-DIALIN-LDT-2998-1
commented on review

Please follow the same naming convention as firmware enums.

4:33 pm

Dara Navaei

More
LEAHI-DIALIN-LDT-3214-1
started review

LEAHI-DIALIN-LDT-3214_[Leahi-Dialin] Refactor to use the generic override functions

11:02 am

Sean Nash

Understand. In this case, the spare positions correspond to bit positions in the FPGA register and the new valves took the 1, 2, and 6 spots.

11:01 am

Zoltan Miskolci

Fixed Typo

11:00 am

Zoltan Miskolci

More
0a01371
changed 1 file
LDT-2004-dialysate-composition-dd, LDT-2004-dry-bicart---dd and 16 more LEAHI-DIALIN-LDT-2998-1

Fixed Typo

- -1
- +1
./rinse_pump.py
- History
- Source
- Diff
./rinse_pump.py

10:57 am

Zoltan Miskolci

Ok (But feels weird to start the counting of existing items from 0 if that's the spare3 instead of 1. Calling the D79 the "zeroeth" spare valve instead of the first spare valve... Personally I thin...

Ok
(But feels weird to start the counting of existing items from 0 if that's the spare3 instead of 1. Calling the D79 the "zeroeth" spare valve instead of the first spare valve... Personally I think we should name them Spare1-8 instead of Spare 0-7)

10:23 am