Fisheye on dgfirmware/firmware/App/Modes/ModeHeatDisinfectActiveCool.h https://devapps.diality.us/changelog/~br%3Drelease%25402fp1_develop/dgfirmware/firmware/App/Modes/ModeHeatDisinfectActiveCool.h Recent activity in "dgfirmware" under directory /firmware/App/Modes/ModeHeatDisinfectActiveCool.h DG-DEN-14763-1: closed by Dara Navaei Tue, 21 Mar 2023 17:12:45 -0700 Dara Navaei https://devapps.diality.us/cru/DG-DEN-14763-1 https://devapps.diality.us/cru/DG-DEN-14763-1 Dara Navaei Closed DG-DEN-14763-1 ]]> DG-DEN-14763-1: reviewers resumed Tue, 21 Mar 2023 08:35:30 -0700 https://devapps.diality.us/cru/DG-DEN-14763-1 https://devapps.diality.us/cru/DG-DEN-14763-1 Sean Nash resumed reviewing DG-DEN-14763-1 DG-DEN-14763_DG HD Dev HD DG Dvt Update Part 10 ]]> DG-DEN-14763-1: comment by Sean Nash "RESOLVED in CODE WALKTHROUGH." Tue, 21 Mar 2023 08:33:20 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16906 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16906 Sean Nash replied to Dara Navaei in DG-DEN-14763-1

RESOLVED in CODE WALKTHROUGH.

]]> DG-DEN-14763-1: comment by Sean Nash "RESOLVED in CODE WALKTHROUGH." Tue, 21 Mar 2023 08:32:48 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16905 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16905 Sean Nash replied to Dara Navaei in DG-DEN-14763-1

RESOLVED in CODE WALKTHROUGH.

]]> DG-DEN-14763-1: comment by Dara Navaei "Done" Tue, 21 Mar 2023 08:30:01 -0700 Dara Navaei https://devapps.diality.us/cru/DG-DEN-14763-1/#c16904 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16904 Dara Navaei replied to Sean Nash in DG-DEN-14763-1

Done

]]> DG-DEN-14763-1: comment by Dara Navaei "Done" Tue, 21 Mar 2023 08:28:22 -0700 Dara Navaei https://devapps.diality.us/cru/DG-DEN-14763-1/#c16903 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16903 Dara Navaei replied to Sean Nash in DG-DEN-14763-1

Done

]]> DG-DEN-14763-1: comment by Sean Nash "This seems wrong." Mon, 20 Mar 2023 23:50:50 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16902 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16902 Sean Nash commented on DG-DEN-14763-1

This seems wrong.

]]> DG-DEN-14763-1: comment by Sean Nash "This seems wrong." Mon, 20 Mar 2023 23:50:13 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16901 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16901 Sean Nash commented on DG-DEN-14763-1

This seems wrong.

]]> DG-DEN-14763-1: reviewers resumed Mon, 20 Mar 2023 20:31:36 -0700 https://devapps.diality.us/cru/DG-DEN-14763-1 https://devapps.diality.us/cru/DG-DEN-14763-1 wbracken  (deleted user) and jtaylor  (deleted user) resumed reviewing DG-DEN-14763-1 DG-DEN-14763_DG HD Dev HD DG Dvt Update Part 10 ]]> DG-DEN-14763-1: comment by jtaylor "RESOLVED in CODE WALKTHROUGH." Mon, 20 Mar 2023 16:34:46 -0700 jtaylor https://devapps.diality.us/cru/DG-DEN-14763-1/#c16899 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16899 jtaylor  (deleted user) replied to Sean Nash in DG-DEN-14763-1

RESOLVED in CODE WALKTHROUGH.

]]> DG-DEN-14763-1: comment by wbracken "RESOLVED IN CODE WALKTHROUGH" Mon, 20 Mar 2023 13:32:35 -0700 wbracken https://devapps.diality.us/cru/DG-DEN-14763-1/#c16898 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16898 wbracken  (deleted user) replied to Dara Navaei in DG-DEN-14763-1

RESOLVED IN CODE WALKTHROUGH

]]> DG-DEN-14763-1: comment by Sean Nash "RESOLVED in CODE WALKTHROUGH." Mon, 20 Mar 2023 09:49:15 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16891 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16891 Sean Nash replied to Dara Navaei in DG-DEN-14763-1

RESOLVED in CODE WALKTHROUGH.

]]> DG-DEN-14763-1: comment by Sean Nash "RESOLVED in CODE WALKTHROUGH." Mon, 20 Mar 2023 09:48:46 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16890 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16890 Sean Nash replied to Dara Navaei in DG-DEN-14763-1

RESOLVED in CODE WALKTHROUGH.

]]> DG-DEN-14763-1: comment by Sean Nash "RESOLVED in CODE WALKTHROUGH." Mon, 20 Mar 2023 09:48:22 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16889 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16889 Sean Nash replied to Dara Navaei in DG-DEN-14763-1

RESOLVED in CODE WALKTHROUGH.

]]> DG-DEN-14763-1: comment by Dara Navaei "Done. Thanks for catching it." Mon, 20 Mar 2023 09:34:18 -0700 Dara Navaei https://devapps.diality.us/cru/DG-DEN-14763-1/#c16885 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16885 Dara Navaei replied to Sean Nash in DG-DEN-14763-1

Done. Thanks for catching it.

]]> DG-DEN-14763-1: comment by Dara Navaei "Done" Mon, 20 Mar 2023 09:28:53 -0700 Dara Navaei https://devapps.diality.us/cru/DG-DEN-14763-1/#c16884 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16884 Dara Navaei replied to Sean Nash in DG-DEN-14763-1

Done

]]> DG-DEN-14763-1: comment by Dara Navaei "Done" Mon, 20 Mar 2023 09:28:17 -0700 Dara Navaei https://devapps.diality.us/cru/DG-DEN-14763-1/#c16883 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16883 Dara Navaei replied to Sean Nash in DG-DEN-14763-1

Done

]]> DG-DEN-14763-1: comment by Sean Nash "Not all of these are disinfect usage (e.g. RO w..." Mon, 20 Mar 2023 08:55:49 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16870 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16870 Sean Nash replied to jtaylor  (deleted user) in DG-DEN-14763-1

Not all of these are disinfect usage (e.g. RO water generated) so I don't think disinfect_usage is appropriate. I am ok with usage_info, though I think DG_USAGE_INFO is better.

]]> DG-DEN-14763-1: comment by Sean Nash "RESOLVED in CODE WALKTHROUGH." Mon, 20 Mar 2023 08:48:18 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16869 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16869 Sean Nash replied to Dara Navaei in DG-DEN-14763-1

RESOLVED in CODE WALKTHROUGH.

]]> DG-DEN-14763-1: comment by Sean Nash "Missing comment?" Mon, 20 Mar 2023 08:43:01 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16868 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16868 Sean Nash commented on DG-DEN-14763-1

Missing comment?

]]> DG-DEN-14763-1: comment by Sean Nash "Everything else in this function should be in a..." Mon, 20 Mar 2023 08:42:12 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16867 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16867 Sean Nash commented on DG-DEN-14763-1

Everything else in this function should be in an else body if given reservoir ID is invalid so we don't try to use an invalid ID.

]]> DG-DEN-14763-1: comment by Sean Nash "Why lower case when others are uppercase?" Mon, 20 Mar 2023 08:36:34 -0700 Sean Nash https://devapps.diality.us/cru/DG-DEN-14763-1/#c16866 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16866 Sean Nash commented on DG-DEN-14763-1

Why lower case when others are uppercase?

]]> DG-DEN-14763-1: comment by Dara Navaei "This is to find the target configuration which ..." Sat, 18 Mar 2023 14:36:22 -0700 Dara Navaei https://devapps.diality.us/cru/DG-DEN-14763-1/#c16865 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16865 Dara Navaei replied to Sean Nash in DG-DEN-14763-1

This is to find the target configuration which in our case it is XDS-110 automatically. It seems the .ccxml file in the targetConfigs folder sets the right debugger and target regardless of this check box being true or false. I set it back to true to be consistent.

]]> DG-DEN-14763-1: comment by Dara Navaei "Done" Fri, 17 Mar 2023 14:42:49 -0700 Dara Navaei https://devapps.diality.us/cru/DG-DEN-14763-1/#c16859 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16859 Dara Navaei replied to wbracken  (deleted user) in DG-DEN-14763-1

Done

]]> DG-DEN-14763-1: comment by jtaylor "I would suggest a more consistent naming for th..." Fri, 17 Mar 2023 08:43:24 -0700 jtaylor https://devapps.diality.us/cru/DG-DEN-14763-1/#c16847 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16847 jtaylor  (deleted user) commented on DG-DEN-14763-1

I would suggest a more consistent naming for the enum items. "DISINFECT_USAGE_" rather than USAGE_INFO_ in the enum elements, and eliminating the "_DIS".
This would be more specific since "USAGE" and "INFO" are redundant, and "DIS" can be confused with "dis-", the Latin prefix.

]]> DG-DEN-14763-1: comment by wbracken "Comment alignment." Thu, 16 Mar 2023 22:08:46 -0700 wbracken https://devapps.diality.us/cru/DG-DEN-14763-1/#c16846 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16846 wbracken  (deleted user) commented on DG-DEN-14763-1

Comment alignment.

]]> DG-DEN-14763-1: comment by wbracken "RESOLVED IN CODE WALKTHROUGH" Thu, 16 Mar 2023 12:12:03 -0700 wbracken https://devapps.diality.us/cru/DG-DEN-14763-1/#c16841 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16841 wbracken  (deleted user) replied to Dara Navaei in DG-DEN-14763-1

RESOLVED IN CODE WALKTHROUGH

]]> DG-DEN-14763-1: comment by wbracken "RESOLVED IN CODE WALKTHROUGH" Thu, 16 Mar 2023 12:11:37 -0700 wbracken https://devapps.diality.us/cru/DG-DEN-14763-1/#c16840 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16840 wbracken  (deleted user) replied to Dara Navaei in DG-DEN-14763-1

RESOLVED IN CODE WALKTHROUGH

]]> DG-DEN-14763-1: comment by wbracken "RESOLVED IN CODE WALKTHROUGH" Thu, 16 Mar 2023 12:10:17 -0700 wbracken https://devapps.diality.us/cru/DG-DEN-14763-1/#c16839 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16839 wbracken  (deleted user) replied to Dara Navaei in DG-DEN-14763-1

RESOLVED IN CODE WALKTHROUGH

]]> DG-DEN-14763-1: comment by wbracken "RESOLVED IN CODE WALKTHROUGH" Thu, 16 Mar 2023 12:08:14 -0700 wbracken https://devapps.diality.us/cru/DG-DEN-14763-1/#c16838 https://devapps.diality.us/cru/DG-DEN-14763-1/#c16838 wbracken  (deleted user) replied to Dara Navaei in DG-DEN-14763-1

RESOLVED IN CODE WALKTHROUGH

]]>