Monitors

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Did you get a chance to test this alarm and uncomment the code?

Did you get a chance to test this alarm and uncomment the code?

Remove empty space.

Remove empty space.

ACToDC?

ACToDC?

Count datatype seems to be integer. Type case to F32 to match with denominator data type.

Count datatype seems to be integer. Type case to F32 to match with denominator data type.

Align comment. Clarify (at least in comments) that these macros are for FPGA count conversions.

Align comment.
Clarify (at least in comments) that these macros are for FPGA count conversions.

Remove extra blank lines.

Remove extra blank lines.

What is 4 (or 2)? Should be #define or enum or const.

What is 4 (or 2)? Should be #define or enum or const.

please align this comments with below structure

please align this comments with below structure

Please align the comments

Please align the comments

could you please align it ?

could you please align it ?

Add blank line (should be 2 before test support banner).

Add blank line (should be 2 before test support banner).

Remove blank line.

Remove blank line.

Needs doxygen comments.

Needs doxygen comments.

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

Reverted change. Added uncompensated conductivity to the conductivity publishing

Reverted change. Added uncompensated conductivity to the conductivity publishing

space after if.

space after if.

Space after if.

Space after if.

If it is useful for current and future conductivity sensor development, you may keep it with TODO comment to remove later once testing is done.

If it is useful for current and future conductivity sensor development, you may keep it with TODO comment to remove later once testing is done.

so, are we removing this? I think it is better to have this as this might make it easier to get and validate the values in future. Not sure what others think.

so, are we removing this? I think it is better to have this as this might make it easier to get and validate the values in future. Not sure what others think.

Remove empty line.

Remove empty line.

Align the indentation

Align the indentation

Align the indentation.

Align the indentation.

Same as above.

Same as above.

Add an event here, when time out happens. Event data should capture fresh side and spent side fill completion status.

Add an event here, when time out happens. Event data should capture fresh side and spent side fill completion status.

You may need to comment/ remove this condition. the pressure stabilized flag should set only when pressure is met (not on chamber fill request)

You may need to comment/ remove this condition. the pressure stabilized flag should set only when pressure is met (not on chamber fill request)