common

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-6200: Updated pre-treatment and prime state enum

DEN-6200: Updated pre-treatment and prime state enum

Let's talk about it. If there's a good reason, fine. But generally I'd like HD and DG to do similar things in the same manner for consistency.

Let's talk about it. If there's a good reason, fine. But generally I'd like HD and DG to do similar things in the same manner for consistency.

On the DG side, I move this function into Alarm Management module. I find it better than individually handle them one by one and it still gives flexibility in case we need to handle another persist...

On the DG side, I move this function into Alarm Management module. I find it better than individually handle them one by one and it still gives flexibility in case we need to handle another persistent alarm differently.

This does not look compatible with changes to PersistentAlarm module. Does this compile? I see this everywhere. I suspect you changed all of these and just need to push main DG branch.

This does not look compatible with changes to PersistentAlarm module. Does this compile? I see this everywhere. I suspect you changed all of these and just need to push main DG branch.

I know these enums are in a header file with "HD" in the name and so the HD_ prefix may seem unnecessary, but I was putting HD_ and DG_ prefixes on mode state enums in case the HD and DG had one or...

I know these enums are in a header file with "HD" in the name and so the HD_ prefix may seem unnecessary, but I was putting HD_ and DG_ prefixes on mode state enums in case the HD and DG had one or more modes with the same name (like Standby) so there would not be a conflict if somebody wanted to include both HD and DG definitions in there module.

HD-DEN-6200_HD Disposable Priming Dialyzer Side
HD-DEN-6200_HD Disposable Priming Dialyzer Side
DG-DEN-6200_HD Disposable Priming Dialyzer Side
DG-DEN-6200_HD Disposable Priming Dialyzer Side
Merge branch 'quick_fix_01' into 2020_EOY_DEMO

DEN-6200: Added alarms for pre-treatment

DEN-6200: Added alarms for pre-treatment

DEN-6200: Added more dialyzer and bypass prime states

DEN-6200: Added more dialyzer and bypass prime states

Merge branch 'Sprint33' into develop

DEN-5963 updated the alarm table

Instead of "if the flag is TRUE", comment should say something like "if disinfect temperature has been reached".

Instead of "if the flag is TRUE", comment should say something like "if disinfect temperature has been reached".

fabs will indicate a leak even if the volume goes up. Is that what you want here?

fabs will indicate a leak even if the volume goes up. Is that what you want here?

Add else for s/w fault. Or maybe load cell driver or Reservoirs module should have a generic get weight function for a given reservoir.

Add else for s/w fault. Or maybe load cell driver or Reservoirs module should have a generic get weight function for a given reservoir.

Change to resEmpty.

Change to resEmpty.

Not a great name. Should be resFull or something like that.

Not a great name. Should be resFull or something like that.

Should be an else (or default if you turn this into a switch) that triggers a s/w fault (bad reservoir param).

Should be an else (or default if you turn this into a switch) that triggers a s/w fault (bad reservoir param).

There should already be an enum for reservoirs somewhere. Use that one.

There should already be an enum for reservoirs somewhere. Use that one.

Should drain pump be turned off?

Should drain pump be turned off?

Is ppiPressure being checked here or in global check I saw at top of exec? Since VPi is closed for much of this mode, the global check is probably not right.

Is ppiPressure being checked here or in global check I saw at top of exec? Since VPi is closed for much of this mode, the global check is probably not right.

Function name is confusing. Change to something like failHeatDisinfection(). Also, this state handler will return DG_HEAT_DISINFECT_STATE_DRAIN_R1 - overriding the complete state being set in your ...

Function name is confusing. Change to something like failHeatDisinfection(). Also, this state handler will return DG_HEAT_DISINFECT_STATE_DRAIN_R1 - overriding the complete state being set in your function.

Should return boolean. Should return FALSE and not request standby if we are not in heat disinfect mode.

Should return boolean. Should return FALSE and not request standby if we are not in heat disinfect mode.

Didn't Sunny say we need at least 25 PSI at inlet?

Didn't Sunny say we need at least 25 PSI at inlet?

I don't see that you're using this status type. Why not just remove it now?

I don't see that you're using this status type. Why not just remove it now?

I would avoid the \ and new line here. If comment won't align, put comment on line above with ///.

I would avoid the \ and new line here. If comment won't align, put comment on line above with ///.

Should this return a boolean? What if we're not in heat disinfect mode when this function is called? Should return FALSE then.

Should this return a boolean? What if we're not in heat disinfect mode when this function is called? Should return FALSE then.