Tasks

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-14344: Air Pump updates. moved to task general. removal of monitor functions. code review changes. Reworked setting motor to be more direct while debugging.

  1. … 3 more files in changeset.
Do we have an alternative to reading if we successfully set the pin high/low?

Do we have an alternative to reading if we successfully set the pin high/low?

Ok. Not sure how HALCoGen got ahead of generated code for a time. RESOLVED in CODE WALKTHROUGH.

Ok. Not sure how HALCoGen got ahead of generated code for a time.
RESOLVED in CODE WALKTHROUGH.

Oh, ok. Then somebody removed a blank line previously and should not have. RESOLVED in CODE WALKTHROUGH.

Oh, ok. Then somebody removed a blank line previously and should not have.
RESOLVED in CODE WALKTHROUGH.

Not sure how we would test pump other than in pre-treatment when we do pressure leak test (bad pump will fail to pressurize venous side). Remove function unless there is some way to test pump direc...

Not sure how we would test pump other than in pre-treatment when we do pressure leak test (bad pump will fail to pressurize venous side). Remove function unless there is some way to test pump directly.

How would you check motor is running? Not clear to me that we need a monitor. Can move broadcast to control exec.

How would you check motor is running? Not clear to me that we need a monitor. Can move broadcast to control exec.

Ok to have this function - just wondering where it would be used? If not needed, remove function and make pump state enum private.

Ok to have this function - just wondering where it would be used? If not needed, remove function and make pump state enum private.

This pin is an output (or it should be), so I don't think a get function makes sense. Where is this function used?

This pin is an output (or it should be), so I don't think a get function makes sense. Where is this function used?

Put constant on right side of compare operator. State should be < num of states.

Put constant on right side of compare operator. State should be < num of states.

Put constant on right side of compare operator. State should be < num of states.

Put constant on right side of compare operator. State should be < num of states.

Remove space before "state".

Remove space before "state".

Remove extra blank lines. Add function headers.

Remove extra blank lines. Add function headers.

Remove extra blank line.

Remove extra blank line.

Add doxygen comment to top of structs and enums.

Add doxygen comment to top of structs and enums.

Align comments.

Align comments.

Remove extra blank line.

Remove extra blank line.

Add blank line after separator.

Add blank line after separator.

What is there to monitor?

What is there to monitor?

Need a doxygen module comment here.

Need a doxygen module comment here.

Add blank line after separator.

Add blank line after separator.

Add blank line before separator.

Add blank line before separator.

This code was auto-generated by HALCoGen to link PIN 36. Did you want me to revert the auto-generated code?

This code was auto-generated by HALCoGen to link PIN 36. Did you want me to revert the auto-generated code?

Remove extra blank line.

Remove extra blank line.

This code was auto-generated by HALCoGen. Did you want me to revert the auto-generated code?

This code was auto-generated by HALCoGen. Did you want me to revert the auto-generated code?

There are 2 available msg IDs above that should be used before adding new ones.

There are 2 available msg IDs above that should be used before adding new ones.

Align comment.

Align comment.

Remove extra blank line.

Remove extra blank line.

Need function header.

Need function header.

Need function header.

Need function header.

What happened here? Why changed?

What happened here? Why changed?