•  

Comment Results

Review Name Created Custom Fields Content
HD-DEN-14175-1 07 Nov 2022

RESOLVED in CODE WALKTHROUGH.

UI-DEN-13962-1 07 Nov 2022

removed spaces.

DG-DEN-14316-1 21 Nov 2022

Alignment

DG-DEN-14197-1 21 Nov 2022

Remove extra blank line.

DG-DEN-14197-1 21 Nov 2022

Remove extra blank line.

DG-DEN-14197-1 21 Nov 2022

Fixed. Thanks!

DG-DEN-14316-1 30 Nov 2022

Alignment

DG-DEN-14356-1 22 Nov 2022

Seems like this is a minimum (not a maximum). Pressure below this level is an alarm which enforces a minimum pressure of 14 PSI.

HD-DEN-14344-3 23 Nov 2022

Fixed. Thanks!

HD-DEN-14344-3 28 Nov 2022

Is this an output file from HalCoGen? If so seems like one shouldn't need to modify this file directly.

HD-DEN-14099-1 22 Nov 2022

please fix naming. comment says arterial but define is calling it as venous.

DG-DEN-14316-1 21 Nov 2022

Alignment

HD-DEN-14344-3 22 Nov 2022

Add blank line before separator.

HD-DEN-14344-3 22 Nov 2022

What is there to monitor?

DG-DEN-14316-1 21 Nov 2022

Remove blank line

DG-DEN-15367-1 11 Jun 2023

Done

DG-DEN-12931-1 04 Aug 2022

Does the broadcast need to be updated with the new struct fields?

HD-DEN-12931-2 22 Jul 2022

Why removed? I thought we needed this for VC?

HD-DEN-12931-2 04 Aug 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-13598-2 26 Sep 2022

Removed.

DG-DEN-12931-1 04 Aug 2022

RESOLVED in CODE WALKTHROUGH.

DG-DEN-15367-1 11 Jun 2023

Done

DG-DEN-12931-1 04 Aug 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-13598-2 27 Sep 2022

RESOLVED in CODE WALKTHROUGH.

DG-DEN-12931-1 04 Aug 2022

RESOLVED in CODE WALKTHROUGH.

DG-DEN-15367-1 06 Jun 2023

Comments.

HD-DEN-13598-2 27 Sep 2022

RESOLVED in CODE WALKTHROUGH.

DIALIN-DEN-13460-1 28 Sep 2022

RESOVLED.

DG-DEN-13834-1 28 Sep 2022

Done.

HD-DEN-13834-1 29 Sep 2022

It is up to date.

HD-DEN-13834-1 12 Oct 2022

Done.

HD-DEN-13834-1 04 Oct 2022

Idea is to re-purpose these (instead of adding new alarms to bottom of enum). We prefer not to collapse the table when an alarm ID goes obsolete so other alarm IDs do not change.

HD-DEN-14001-1 04 Oct 2022

Sort of. You will get something between the ms specified and 1 ms less because you don't know where you are in a given ms when you get your start time.
Recommend changing this to 2 ms timeout.

HD-DEN-13903-2 12 Oct 2022

This is an older style of persistence? line 214 shows that airTrapIllegalLevelSensorsCtr gets decremented if it is greater than 0.

DG-DEN-13834-1 12 Oct 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-14000-1 10 Oct 2022

updated to use pointer.

HD-DEN-14000-1 10 Oct 2022

Updated to define, added comment.

DIALIN-DEN-13903-1 12 Oct 2022

Fixed. Thanks!

DIALIN-DEN-13903-1 12 Oct 2022

RESOLVED.

DG-DEN-13834-1 12 Oct 2022

Done.

HD-DEN-13834-1 12 Oct 2022

Done.

HD-DEN-14000-1 12 Oct 2022

Remove extra blank line.

HD-DEN-14000-1 07 Oct 2022

I think getting here would maybe indicate flush/disinfect ended naturally or faulted out while waiting for confirmation. I think appropriate thing to do is to send a reject back to UI? No alarm.

DG-DEN-13834-1 13 Oct 2022

Removed the code.

HD-DEN-13834-1 17 Oct 2022

Removed it.

HD-DEN-13834-1 17 Oct 2022

RESOLVED IN CODE WALKTHROUGH

HD-DEN-14344-3 22 Nov 2022

Remove extra blank line.

DG-DEN-14197-1 21 Nov 2022

Remove blank line.

HD-DEN-14150-1 18 Oct 2022

Updated the function header to include dialysateInPumpRotorCounter. and dipStopAtHomePosition as output.

DG-DEN-13834-1 18 Oct 2022

RESOLVED in CODE WALKTHROUGH.