Pressures.c

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-13834 added the drain pump current and direction alarms. Updated the heaters driver disinfect control. Added a check to ignore the trimmer heater stop command from HD if DG is in the heat disinfect or chemical disinfect

  1. … 6 more files in changeset.
DEN-13834 added more self test alarms. Added FPGA functions to get the drain pump current feedbakc, speed feedback, and direction

  1. … 5 more files in changeset.
updated pressure offset to reflect new calculations.

Merge branch 'DEN-13903-s80-mg-dg-comm-buffer-opti' into develop

DEN-13903: updated pressure offset for psia conversion

Merge branch 'DEN-13903-s80-mg-dg-comm-buffer-opti' into develop

DEN-13903: DEN-13987: Updated dg pressure's adc to psi to reflect new calculations.

Updated Pressure adc count to psi to reflect new calculations.

Alarm for invalid loadCellID?

Alarm for invalid loadCellID?

Should there be an alarm if loadCellID is invalid?

Should there be an alarm if loadCellID is invalid?

Why the change?

Why the change?

DEN-13834 added self test alarms

  1. … 6 more files in changeset.
Do we still need this? If so, can we make a s/w config instead?

Do we still need this? If so, can we make a s/w config instead?

Is this resolved?

Is this resolved?

Why are we not checking payload length?

Why are we not checking payload length?

If this function is not yet needed, should we remove or at least put a #ifdef PHASE_1B build flag around it? Otherwise this is dead code right now.

If this function is not yet needed, should we remove or at least put a #ifdef PHASE_1B build flag around it? Otherwise this is dead code right now.

Why are we not enforcing payload length check?

Why are we not enforcing payload length check?

Can this declaration be moved to top of function?

Can this declaration be moved to top of function?

Why are we hiding this condition from Vectorcast?

Why are we hiding this condition from Vectorcast?

This comment should precede the if statement above.

This comment should precede the if statement above.

Is this 109?

Is this 109?

This looks like 104 (not 106). And we should be noting every 5th enum anyway.

This looks like 104 (not 106). And we should be noting every 5th enum anyway.

Is this resolved now? Can we un-comment this code?

Is this resolved now? Can we un-comment this code?

Remove isPOSTComplete.

Remove isPOSTComplete.

Can we remove "TODO" since implementation is not required at this time. Re-phrase comment to something like "If temp sensors require calibration, implement here."

Can we remove "TODO" since implementation is not required at this time. Re-phrase comment to something like "If temp sensors require calibration, implement here."

POST does nothing? If always in progress, do we get stuck in this test?

POST does nothing? If always in progress, do we get stuck in this test?

Remove blank line.

Remove blank line.

Can these declarations be moved to top of function?

Can these declarations be moved to top of function?

Lets get this question answered. I don't see the harm in having a max flow. I can't imagine we're ever seeing that much flow, so not sure why this has to be commented out while we sort out its exis...

Lets get this question answered. I don't see the harm in having a max flow. I can't imagine we're ever seeing that much flow, so not sure why this has to be commented out while we sort out its existence.

Remove extra blank line.

Remove extra blank line.