This is a list of all comments for DIALIN-DEN-6372-1. Review Summary: No summary ---------------------------------------- File: dialin/ui/hd_simulator.py Revision Comment by pmontazemi on 15 February 2021, 13:21 https://devapps.diality.us/cru/DIALIN-DEN-6372-1#c8113 Remove extra lines. Reply by Sean Nash on 16 February 2021, 10:56 > Done. Reply by pmontazemi on 16 February 2021, 13:07 > RESOLVED. ---------------------------------------- File: tests/test_uf.py Revision Comment by pmontazemi on 15 February 2021, 13:22 https://devapps.diality.us/cru/DIALIN-DEN-6372-1#c8114 I would recommend using the following naming convention: hdPress instead of hdPres, which can be confused with HD present (hdPres). Reply by Sean Nash on 16 February 2021, 10:58 > Done. Reply by pmontazemi on 16 February 2021, 13:08 > RESOLVED. --- ID: DIALIN-DEN-6372-1 https://devapps.diality.us/cru/DIALIN-DEN-6372-1 Title: DIALIN-DEN-6372_HD Rinseback Statement of Objectives: State: Closed Summary: Author: Sean Nash Reviewers: (0 active, 2 completed*) plucia (*) pmontazemi (*)