Modes

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done (By Darren).

Done (By Darren).

Done.

Done.

Done.

Done.

This turns off the trimmer hearter, to smooth heater control (as before). The switch state (next up) actually makes the reservoir switch, the next state.

This turns off the trimmer hearter, to smooth heater control (as before). The switch state (next up) actually makes the reservoir switch, the next state.

This works, even if the s.t. dry test is skipped.

This works, even if the s.t. dry test is skipped.

Done (By Darren)

Done (By Darren)

The normal value is too tight for the current hardware; it always fails this test, without the wider volume tolerance.

The normal value is too tight for the current hardware; it always fails this test, without the wider volume tolerance.

Done.

Done.

Done (by Darren)

Done (by Darren)

Comment Alignment

Comment Alignment

Add pendingUFVolumeChange, pressTreatmentTimeSecs, pendingParamChangesTimer,

Add pendingUFVolumeChange, pressTreatmentTimeSecs, pendingParamChangesTimer,

Fixed. Thanks!

Fixed. Thanks!

Use SEC_PER_MIN instead of 60. Also comment says min, but really it is ms.

Use SEC_PER_MIN instead of 60. Also comment says min, but really it is ms.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

it was a double include anyway. Fixed. Thanks!

it was a double include anyway. Fixed. Thanks!

Alphabetical ordering.

Alphabetical ordering.

This is not going to work. Revisit. See code review for DEN-14749 for correct arrangement.

This is not going to work. Revisit. See code review for DEN-14749 for correct arrangement.

HD-DEN-14749_Syringe Pump Plunger Remains Extended During Pretreatment Causing HD Software Fault
HD-DEN-14749_Syringe Pump Plunger Remains Extended During Pretreatment Causing HD Software Fault
HD-DEN-14763_DG HD Dev HD DG Dvt Update Part 10
HD-DEN-14763_DG HD Dev HD DG Dvt Update Part 10
RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

This math no longer works because we changed the displacement time from 1 minute (which allowed for this simplified approach) to 30 seconds. Generalize the math. Suggested approach: -integrate vol...

This math no longer works because we changed the displacement time from 1 minute (which allowed for this simplified approach) to 30 seconds.
Generalize the math.

Suggested approach:
-integrate volume above properly (don't just sum rates over time).
-Should be: fmdIntegratedVolume += ( getMeasuredDialInFlowRate() / (F32)( ( SEC_PER_MIN * MS_PER_SECOND ) / TASK_GENERAL_INTERVAL ) );
-Then we don't need to convert here at end of displacement - can just use fmdIntegratedVolume as is in verify state.