Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DIAL-258: added clear of 4th level state once we finish wet self tests.

fixed. set to no sub state (0xFF) once we transition to pretx recirc

fixed. set to no sub state (0xFF) once we transition to pretx recirc

Looks like this is only being set in wet self-tests substate of pre-tx prime. Once we leave wet self-tests, should this 4th level state be set back to zero or something?

Looks like this is only being set in wet self-tests substate of pre-tx prime. Once we leave wet self-tests, should this 4th level state be set back to zero or something?

Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

Merge branch 'DEN-15317-usability-hd-door-open-alarm-during-post-treatment' into develop

DEN-15317: Added alarm re-trigger blocked flag to alarm status broadcast msg.

DEN-15317: Moved post-tx homing to ensure it happens in all cases.

Seems like we should be calling CPLD function here and in Fans, DialInFlow, and DGInterface too. Also recommend calling the isACPowerLost() function in just one place - maybe alarm mgmt exec to gi...

Seems like we should be calling CPLD function here and in Fans, DialInFlow, and DGInterface too.

Also recommend calling the isACPowerLost() function in just one place - maybe alarm mgmt exec to give you your 10 sec alarm block. Maybe rename the function to checkACPowerLost() with no return value and make it static (private).

It is a timer counter. Counts down to zero. And still need to address function header per Bill's first comment.

It is a timer counter. Counts down to zero.

And still need to address function header per Bill's first comment.

This seems unnecessary now with your new function doing this all the time. Consider removing.

This seems unnecessary now with your new function doing this all the time. Consider removing.

This is technically a timer counter based on task tick of 50ms.

This is technically a timer counter based on task tick of 50ms.

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

How is this different than checking CPLD? Condition is cleared when CPLD no longer sees power loss (w/ 150ms delay I guess). Seems simpler to use CPLD function instead of checking 2 alarms.

How is this different than checking CPLD? Condition is cleared when CPLD no longer sees power loss (w/ 150ms delay I guess). Seems simpler to use CPLD function instead of checking 2 alarms.

Remove extra blank line.

Remove extra blank line.

DEN-15317: Fixed issue with failure to populate pressure limit state in the pressure data broadcast msg x9.

Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

DEN-15317: Addressed saline bolus issue as part of DEN-15359.

HD-DEN-15317_Usability HD Door Open Alarm During Post Treatment
HD-DEN-15317_Usability HD Door Open Alarm During Post Treatment
Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

Merge branch 'DEN-15317-usability-hd-door-open-alarm-during-post-treatment' into develop