This is a list of all comments for UI-DEN-11980-1.

Review Summary: No summary


----------------------------------------
File: sources/model/MModel.h
Revision Comment by Behrouz NematiPour on 16 February 2022, 10:00 
https://devapps.diality.us/cru/UI-DEN-11980-1#c11937

  Please 
   - Everywhere, 
   - BloodFlow should come before Dialysate.
  Includes, .pro file, ...

  Reply by Michael Garthwaite on 16 February 2022, 12:51
  > Fixed. Thanks!

    Reply by Behrouz NematiPour on 16 February 2022, 18:30
    > RESOLVED.

----------------------------------------
File: sources/model/hd/data/treatment/MTreatmentBloodFlowData.cpp
Revision Comment by Behrouz NematiPour on 16 February 2022, 09:36 
https://devapps.diality.us/cru/UI-DEN-11980-1#c11923

  remove this comment since it was there to separate the two
  models.

  Reply by Behrouz NematiPour on 16 February 2022, 18:32
  > RESOLVED.

Revision Comment by Behrouz NematiPour on 16 February 2022, 09:36 
https://devapps.diality.us/cru/UI-DEN-11980-1#c11924

  align the {code}.value{code}

  Reply by Michael Garthwaite on 16 February 2022, 12:52
  > Fixed. Thanks!

    Reply by Behrouz NematiPour on 16 February 2022, 18:32
    > RESOLVED.

Revision Comment by Behrouz NematiPour on 16 February 2022, 09:33 
https://devapps.diality.us/cru/UI-DEN-11980-1#c11920

  Align with the reset.

  Reply by Michael Garthwaite on 16 February 2022, 12:52
  > Fixed. Thanks!

    Reply by Behrouz NematiPour on 16 February 2022, 18:34
    > Please from the equal sign ( = )

      Reply by Michael Garthwaite on 17 February 2022, 08:46
      > Fixed. Thanks!

        Reply by Behrouz NematiPour on 17 February 2022, 09:47
        > RESOLVED

----------------------------------------
File: sources/model/hd/data/treatment/MTreatmentBloodFlowData.h
Revision Comment by Behrouz NematiPour on 16 February 2022, 09:39 
https://devapps.diality.us/cru/UI-DEN-11980-1#c11925

  change #7 to #8

  Reply by Michael Garthwaite on 16 February 2022, 12:52
  > Fixed. Thanks!

    Reply by Behrouz NematiPour on 16 February 2022, 18:31
    > RESOLVED.

Revision Comment by Behrouz NematiPour on 16 February 2022, 09:43 
https://devapps.diality.us/cru/UI-DEN-11980-1#c11928

  please update the comment.

  Reply by Michael Garthwaite on 16 February 2022, 12:52
  > Fixed. Thanks!

    Reply by Behrouz NematiPour on 16 February 2022, 18:32
    > RESOLVED.

----------------------------------------
File: sources/model/hd/data/treatment/MTreatmentInletFlowData.cpp
Revision Comment by Behrouz NematiPour on 16 February 2022, 09:36 
https://devapps.diality.us/cru/UI-DEN-11980-1#c11922

  remove this comment since it was there to separate the two
  models.

  Reply by Michael Garthwaite on 16 February 2022, 12:51
  > Fixed. Thanks!

    Reply by Behrouz NematiPour on 16 February 2022, 18:33
    > RESOLVED.

----------------------------------------
File: sources/model/hd/data/treatment/MTreatmentInletFlowData.h
Revision Comment by Behrouz NematiPour on 16 February 2022, 09:30 
https://devapps.diality.us/cru/UI-DEN-11980-1#c11917

  Please change name of the file(s) to:
  MTreatmentDialysateFlowData.[h,cpp]
  Inlet => Dialysate

  Reply by Michael Garthwaite on 16 February 2022, 12:51
  > Fixed. Thanks!

    Reply by Behrouz NematiPour on 16 February 2022, 18:34
    > RESOLVED.

----------------------------------------
File: sources/view/hd/data/treatment/VTreatmentBloodFlow.cpp
Revision Comment by Behrouz NematiPour on 16 February 2022, 09:45 
https://devapps.diality.us/cru/UI-DEN-11980-1#c11929

  Pleae align the );

  Reply by Michael Garthwaite on 16 February 2022, 12:52
  > Fixed. Thanks!

    Reply by Behrouz NematiPour on 16 February 2022, 18:31
    > RESOLVED.


---

ID: UI-DEN-11980-1 https://devapps.diality.us/cru/UI-DEN-11980-1

Title: UI-DEN-11980_SW Dev Sprint 64
Statement of Objectives:


State: Closed

Summary: 

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