This is a list of all comments for DIALIN-DEN-7117-1.

Review Summary: No summary


----------------------------------------
File: dialin/hd/syringe_pump.py
Revision Comment by plucia on 24 March 2021, 16:27 
https://devapps.diality.us/cru/DIALIN-DEN-7117-1#c8770

  These are defined in ui/hd_simulator.py as well. Common is
  probably the best place to put them. An enum would be good as
  Quang mentioned

  Reply by Sean Nash on 24 March 2021, 23:37
  > Moved to common and converted to enum.

    Reply by plucia on 01 April 2021, 09:13
    > RESOLVED

Revision Comment by qnguyen on 24 March 2021, 10:35 
https://devapps.diality.us/cru/DIALIN-DEN-7117-1#c8700

  Consider turning these into Enums.

  Reply by Sean Nash on 06 April 2021, 14:39
  > Done.

    Reply by qnguyen on 06 April 2021, 14:39
    > RESOLVED.

----------------------------------------
File: dialin/dg/load_cells.py
Revision Comment by qnguyen on 24 March 2021, 10:34 
https://devapps.diality.us/cru/DIALIN-DEN-7117-1#c8698

  Remove test code.

  Reply by Sean Nash on 24 March 2021, 23:36
  > Done.

    Reply by qnguyen on 26 March 2021, 10:35
    > RESOLVED.

----------------------------------------
File: tests/test_syringe.py
Revision Comment by pmontazemi on 07 April 2021, 09:39 
https://devapps.diality.us/cru/DIALIN-DEN-7117-1#c9045

  Align with the rest.

  Reply by Sean Nash on 07 April 2021, 09:55
  > Done.

    Reply by pmontazemi on 13 April 2021, 14:51
    > RESOLVED.


---

ID: DIALIN-DEN-7117-1 https://devapps.diality.us/cru/DIALIN-DEN-7117-1

Title: DIALIN-DEN-7117_HD Heparin Delivery
Statement of Objectives:


State: Closed

Summary: 

Author: Sean Nash
Moderator: Sean Nash
Reviewers: (0 active, 3 completed*)
     qnguyen   (*)
     plucia   (*)
     pmontazemi   (*)