This is a list of all comments for HD-DEN-15023-1.

Review Summary: No summary


----------------------------------------
File: firmware/App/Controllers/SyringePump.c
Revision Comment by Sean Nash on 01 May 2023, 10:45 
https://devapps.diality.us/cru/HD-DEN-15023-1#c17284

  This appears to be in ms.  Should at least say in comment.

  Reply by Michael Garthwaite on 01 May 2023, 11:07
  > Fixed. Thanks!

    Reply by Sean Nash on 01 May 2023, 11:42
    > RESOLVED in CODE WALKTHROUGH.

Revision Comment by Sean Nash on 01 May 2023, 10:49 
https://devapps.diality.us/cru/HD-DEN-15023-1#c17285

  Should this be looking at >2.5V (instead of pre-tx
  occlusionDetected flag)?

  Reply by Michael Garthwaite on 01 May 2023, 11:06
  > Fixed. Thanks!

    Reply by Sean Nash on 01 May 2023, 11:43
    > RESOLVED in CODE WALKTHROUGH.

----------------------------------------
File: firmware/App/Modes/ModeTreatmentParams.c
Revision Comment by Sean Nash on 01 May 2023, 10:50 
https://devapps.diality.us/cru/HD-DEN-15023-1#c17286

  Remove extra blank line.

  Reply by Michael Garthwaite on 01 May 2023, 11:07
  > Fixed. Thanks!

    Reply by Sean Nash on 01 May 2023, 11:42
    > RESOLVED in CODE WALKTHROUGH.

Revision Comment by Sean Nash on 01 May 2023, 10:51 
https://devapps.diality.us/cru/HD-DEN-15023-1#c17287

  Add blank line between ifs.

  Reply by Michael Garthwaite on 01 May 2023, 11:07
  > Fixed. Thanks!

    Reply by Sean Nash on 01 May 2023, 11:42
    > RESOLVED in CODE WALKTHROUGH.


---

ID: HD-DEN-15023-1 https://devapps.diality.us/cru/HD-DEN-15023-1

Title: HD-DEN-15023_Variation IN Syringe Pump Position Between Dvt Units
Statement of Objectives:


State: Closed

Summary: 

Author: Michael Garthwaite
Moderator: Michael Garthwaite
Reviewers: (4 active, 2 completed*)
     Sean Nash   (*)
     wbracken   (*)
     Dara Navaei
     Darren Cox
     jtaylor
     Steve Jarpe