•  

Comment Results

Review Name Created Custom Fields Content
HD-DEN-8679-1 27 Jul 2021

The first sizeof() has ALARM_ID_T. You converted that param to a U32 (which is appropriate) and so you should have U32 in that sizeof().

DG-DEN-11114-1 04 Jan 2022

Remove extra blank line.

HD-DEN-8679-1 29 Jul 2021

RESOLVED in CODE WALKTHROUGH.

DG-DEN-7568-1 23 Apr 2021

Done.

DG-DEN-7568-1 21 Apr 2021

Align or remove extra space.

DG-DEN-9054-1 27 Jul 2021

Put test functions at bottom of module (after usual banner).

DG-DEN-7568-1 23 Apr 2021

RESOLVED in CODE WALKTHROUGH.

DG-DEN-11928-1 01 Mar 2022

Reverted the code with timeout. Need to add timeout requirement in the Inlet Water Check in the DG SRS.

HD-DEN-7347-1 25 Apr 2021

This function does not return anything.

DG-DEN-9054-1 29 Jul 2021

Done.

DIALIN-DEN-8030-1 30 Jul 2021

RESOLVED.

DIALIN-DEN-11980-1 16 Feb 2022

struct.py lists the args to unpack as such:

"The remaining chars indicate types of args and must match exactly;
these can be preceded by a decimal repeat count:
x: pad byte (no data);
c:char;
b:signed byte;
B:unsigned byte;
?: _Bool (requires C99; if not available, char is used instead)
h:short;
H:unsigned short;
i:int;
I:unsigned int; (Capital i)
l:long;
L:unsigned long;
f:float;
d:double;
e:half-float."

Rotor_count is sent from the HD as an unsigned int (U32).

DG-DEN-9054-1 30 Jul 2021

Should we debounce switches before changing states? Check with Noe to see if FPGA is already debouncing these signals.

UI-DEN-9411-1 12 Aug 2021

It has only one parameter and is 15 bytes long.
By definition, it is different than the others.

0x9900 153 0x020 Rsp Y HD UI HD serial number ASCII text - 15 bytes, not null term
0x8800 136 0x070 Rsp Y DG UI DG serial number ASCII text - 15 bytes, not null term
UI-DEN-9411-1 12 Aug 2021

Missing brief.

DG-DEN-11928-1 01 Mar 2022

Removed the lines
requestConcentratePumpOff( CONCENTRATEPUMPS_CP1_ACID );
requestConcentratePumpOff( CONCENTRATEPUMPS_CP2_BICARB );

The concentrate pumps are already OFF prior to transitioning to produce dialysate state.

HD-DEN-11098-1 04 Jan 2022

Yes, see
#ifdef SKIP_PRIMING
currentPrimeState = HD_PRIME_RESERVOIR_ONE_FILL_COMPLETE_STATE;
#else
currentPrimeState = HD_PRIME_WAIT_FOR_USER_START_STATE;
#endif

HD-DEN-11098-1 04 Jan 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11098-1 04 Jan 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11098-1 04 Jan 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11098-1 04 Jan 2022

RESOLVED in CODE WALKTHROUGH.

UI-DEN-12121-1 02 Mar 2022

Required to display the Alarm Top on the diagnostic screen

HD-DEN-10602-1 04 Jan 2022

RESOLVED in CODE WALKTRHOUGH.

HD-DEN-11250-1 31 Jan 2022

Why do we need the second part of this condition? When we start wet self tests, bolus volume delivered should be zero.

DIALIN-UI-S63-1 15 Feb 2022

RESOLVED

UI-DEN-12121-1 02 Mar 2022

Fixed. Thanks!

HD-DEN-11250-1 15 Feb 2022

Remove this line. This change was done in DEN-11750 branch.

HD-DEN-11250-1 01 Feb 2022

Not sure why we need stopPump flag - can't we just call the check function w/ no parameters?

UI-DEN-12121-1 03 Mar 2022

RESOLVED.

DG-DEN-11928-1 01 Mar 2022

I forgot the pumps run at 48 mL/min not 48 mL/sec.

total line volume + 25% = 60mL + 25% of 60mL = 75 mL.

Concentrate pumps run at 48 mL/ min.

( 75 mL / 48 mL ) * 60 seconds = 93.75 seconds (used 95 seconds for PRIME_CONCENTRATE_LINES_TIME_OUT_MS)

DG-DEN-11928-1 03 Mar 2022

Changed the PRIME_CONCENTRATE_LINES_TIME_OUT_MS to 95 seconds.

DG-DEN-11750-1 03 Mar 2022

RESOVLED IN CODE WALKTHROUGH.

HD-DEN-11750-2 03 Mar 2022

RESOVLED IN CODE WALKTHROUGH.

HD-DEN-11750-2 03 Mar 2022

RESOVLED IN CODE WALKTHROUGH.

HD-DEN-11750-2 03 Mar 2022

RESOVLED IN CODE WALKTHROUGH.

HD-DEN-11750-2 02 Mar 2022

Fix indenting here.

DG-DEN-11928-1 23 Mar 2022

Removed
1. static BOOL checkDialysateTemperature( void ) declaration
2. ( TRUE == checkDialysateTemperature() ) reference
3. static BOOL checkDialysateTemperature( void ) definition

DG-DEN-11928-1 30 Mar 2022

Indent comment properly.

DIALIN-DEN-11928-1 28 Mar 2022

Please update the documentation.

DG-DEN-11928-1 30 Mar 2022

Is this alarm really logged to treatment log?

UI-DEN-11980-1 16 Feb 2022

Pleae align the );

HD-DEN-8679-1 27 Jul 2021

I think the data are floats, so 0 should be 0.0.

HD-DEN-7395-1 19 Apr 2021

Keep the function and turn TRUE into SELF_TEST_STATUS_PASSED.

HD-DEN-11980-1 16 Feb 2022

Check that Dialin has logged in before acting on a Dialin request (TRUE == isTestingActivated()).

HD-DEN-7395-1 19 Apr 2021

RESOLVED in CODE WALKTHROUGH.

HD-DEN-8679-1 29 Jul 2021

RESOLVED in CODE WALKTHROUGH.

HD-DEN-7395-1 19 Apr 2021

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11980-1 16 Feb 2022

Check editor settings. Tabs should be set to 4 spaces.

HD-DEN-7395-1 19 Apr 2021

RESOLVED in CODE WALKTHROUGH.

HD-DEN-8679-1 29 Jul 2021

RESOLVED in CODE WALKTRHOUGH.