This is a list of all comments for DIALIN-DEN-14197-1. Review Summary: No summary ---------------------------------------- File: dialin/dg/dialysate_fill.py Revision Comment by Sean Nash on 29 December 2022, 09:53 https://devapps.diality.us/cru/DIALIN-DEN-14197-1#c15789 Function name and brief suggest we are overriding fill target volume but message enum suggests we are overriding the integrated volume from flow that we use to check load cell vs. flow sensor for each fill. I think latter is correct. Should rename function and update brief to clarify purpose. Reply by Dara Navaei on 02 January 2023, 17:16 > Done Reply by Sean Nash on 03 January 2023, 08:44 > RESOLVED. --- ID: DIALIN-DEN-14197-1 https://devapps.diality.us/cru/DIALIN-DEN-14197-1 Title: DIALIN-DEN-14197_SW S83 MG Dial 212 Dial 200 Statement of Objectives: State: Closed Summary: Author: Michael Garthwaite Moderator: Michael Garthwaite Reviewers: (1 active, 2 completed*) Sean Nash (*) Dara Navaei (*) Behrouz NematiPour