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

Review Summary: No summary


----------------------------------------
File: firmware/App/Modes/ModePostTreat.c
Revision Comment by wbracken on 12 April 2023, 10:44 
https://devapps.diality.us/cru/HD-DEN-15105-1#c17076

  Update header

  Reply by Sean Nash on 12 April 2023, 12:01
  > Updated.

    Reply by wbracken on 12 April 2023, 17:27
    > RESOLVED IN CODE WALKTHROUGH

----------------------------------------
File: firmware/App/Modes/ModeInitPOST.c
Revision Comment by wbracken on 13 April 2023, 12:31 
https://devapps.diality.us/cru/HD-DEN-15105-1#c17087

  Should be dgPOSTPassed.

  Reply by Sean Nash on 13 April 2023, 13:50
  > Fixed.

    Reply by wbracken on 13 April 2023, 13:58
    > RESOLVED IN CODE WALKTHROUGH

----------------------------------------
File: firmware/App/Modes/Dialysis.c
Revision Comment by wbracken on 13 April 2023, 12:30 
https://devapps.diality.us/cru/HD-DEN-15105-1#c17086

  Update header.

  Reply by Sean Nash on 13 April 2023, 13:52
  > Fixed.

    Reply by wbracken on 13 April 2023, 13:58
    > RESOLVED IN CODE WALKTHROUGH

----------------------------------------
File: firmware/App/Modes/ModePreTreat.c
Revision Comment by wbracken on 13 April 2023, 14:03 
https://devapps.diality.us/cru/HD-DEN-15105-1#c17109

  Update header.

  Reply by Sean Nash on 14 April 2023, 08:29
  > What update is required?

    Reply by wbracken on 14 April 2023, 09:07
    > Variables in lines 153 to 165 should be in header... unless
    > pre-treatment mode module initialized covers them.

      Reply by Sean Nash on 14 April 2023, 09:21
      > For init functions, I typically do not itemize variables
      > (list too long) and just say initialize module.

        Reply by wbracken on 14 April 2023, 09:23
        > RESOLVED IN CODE WALKTHROUGH

Revision Comment by wbracken on 13 April 2023, 14:04 
https://devapps.diality.us/cru/HD-DEN-15105-1#c17110

  Update header

  Reply by Sean Nash on 14 April 2023, 08:28
  > Fixed.

    Reply by wbracken on 14 April 2023, 09:13
    > RESOLVED IN CODE WALKTHROUGH

Revision Comment by wbracken on 13 April 2023, 14:05 
https://devapps.diality.us/cru/HD-DEN-15105-1#c17111

  Update header, setUFVolStatus

  Reply by Sean Nash on 14 April 2023, 08:27
  > Fixed.

    Reply by wbracken on 14 April 2023, 09:12
    > RESOLVED IN CODE WALKTHROUGH


---

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

Title: HD-DEN-15105_Dialyzer Flip
Statement of Objectives:


State: Closed

Summary: 

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