FlushPermeate.c

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merged latest staging into develop

  1. … 33 more files in changeset.
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 121 more files in changeset.
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 19 more files in changeset.
Resolved merge conflicts

  1. … 19 more files in changeset.
Resolved

Resolved

LDT-2936 fixed the macro comments

Change comment from 3 to 5 minutes to match code.

Change comment from 3 to 5 minutes to match code.

I don't see fix

I don't see fix

Fixed the ordering

Fixed the ordering

LDT-2936 fixed permeate overfill issue and added tank controls to verify state

  1. … 5 more files in changeset.
LDT-2936-iofp RO permeate flush timeout LDT-1755

I'm confused as to why the available alarm that this alarm replaced is in a different row within this table. That seems wrong.

I'm confused as to why the available alarm that this alarm replaced is in a different row within this table. That seems wrong.

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 121 more files in changeset.
LDT-2936 address review comments to add spacing in if statements

  1. … 4 more files in changeset.
Handled in FPOperationModes and respective states of PreGen have it.

Handled in FPOperationModes and respective states of PreGen have it.

Checked with Eliza if they need it for alarm persistence time. Not needed at this time, we can add them in future

Checked with Eliza if they need it for alarm persistence time. Not needed at this time, we can add them in future

Fixed. Thanks

Fixed. Thanks

Fixed.

Fixed.

Fixed.

Fixed.

Fixed. thanks

Fixed. thanks

I don't think the number of samples is what you want as 3rd parameter. You want number of bytes so 4 x number of samples so you should pass "sizeof( roRRSamples )".

I don't think the number of samples is what you want as 3rd parameter. You want number of bytes so 4 x number of samples so you should pass "sizeof( roRRSamples )".

Put space between if and ( for all conditions in this function.

Put space between if and ( for all conditions in this function.

Put first condition in () too.

Put first condition in () too.

Small letter used

Small letter used

LDT-2936 added permeate flush exit criteria. Resolved compile errors

  1. … 5 more files in changeset.
Resolved.

Resolved.

Resolved. Thanks

Resolved. Thanks

We might need to provide a possibility to override the persistence time on alarm. To be clarified.

We might need to provide a possibility to override the persistence time on alarm. To be clarified.