common

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-7395: Removed Detector from names of functions

Addressed.

Addressed.

DEN-7395: Lumped Blood Leak Detector status and state into one published data

Addressed.

Addressed.

Merge branch 'staging' into develop

Comments shall start with capital letter.

Comments shall start with capital letter.

Replace a with a more meaningful variable name.

Replace a with a more meaningful variable name.

Align comment.

Align comment.

Align comment.

Align comment.

Add TODO or delete.

Add TODO or delete.

Replace logic with a temp parameter (call it whatever you like) and then use that parameter in the function call. This will make the code a lot more legible.

Replace logic with a temp parameter (call it whatever you like) and then use that parameter in the function call. This will make the code a lot more legible.

Why not using a union? Much more optimized for memory-limited devices.

Why not using a union? Much more optimized for memory-limited devices.

Where did these go?

Where did these go?

Remove "for" at end of @param sentence.

Remove "for" at end of @param sentence.

Remove "for" at end of @param sentence.

Remove "for" at end of @param sentence.

Why using structs instead of enums?

Why using structs instead of enums?

...operating temperatures between 5 degrees C and 95 degrees C.

...operating temperatures between 5 degrees C and 95 degrees C.

...operating temperatures between 5 degrees C and 95 degrees C.

...operating temperatures between 5 degrees C and 95 degrees C.

Why commented out?

Why commented out?

If both of these functions are using the same constant, why do we have two functions? Use just one (unless it is planned for them to inherit different constants down the road).

If both of these functions are using the same constant, why do we have two functions? Use just one (unless it is planned for them to inherit different constants down the road).

Why commented out?

Why commented out?

Spell out P in variable name (or use "Press" at least).

Spell out P in variable name (or use "Press" at least).

Did we want to mention "set" in the two variables' name?

Did we want to mention "set" in the two variables' name?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Need to add state machine in this function. Call state handler functions from a switch statement.

Need to add state machine in this function. Call state handler functions from a switch statement.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Preferred architecture is to just set flag here and then let your state machine see the flag and change the state and command the fpga when it next executes.

Preferred architecture is to just set flag here and then let your state machine see the flag and change the state and command the fpga when it next executes.

Preferred architecture is to just set a request flag here and then let your state machine see the flag and change the state and command the FPGA action the next time it executes.

Preferred architecture is to just set a request flag here and then let your state machine see the flag and change the state and command the FPGA action the next time it executes.

getCurrentOperationMode() returns the current operation mode (standby, pre-treatment, treatment, post-treatment, etc.), not the sub-mode.

getCurrentOperationMode() returns the current operation mode (standby, pre-treatment, treatment, post-treatment, etc.), not the sub-mode.