I guess Eliza mentioned not to move it to temperature class/module unless otherwise it has own Part ID in flow path (like conductivity sensor temperature).
Please put a comment here for next developer to avoid confusion: /* It is only for the dry-demo due to its independent flow. Otherwise, won't needed in the normal designed user-flow. */
Is there a specific reason for 0.600 being the default value? If this is coming from Denali, we should review the value; otherwise, it needs to be discussed, reviewed, and a comment added to clarify.
Radu: "The Team is in the progress of updating and refactoring. and classes and functions may need to change to better be able to test. Therefore all the missing documentation will be addressed in upcoming build"
I don't think we want to use recorded volume (would want inactive reservoir if we did). I think we want to use your function here that calculates the target fill volume.
You are running release
CR4.8.14
FE4.8.14
(20240111091859 2024-01-11 09:20),
please report your release number when reporting bugs.
