These are Denali valves. We need to set all Leahi TD actuators here as appropriate. This will eventually be specified by system arch document in JAMA if it's not already. If not yet, let's do the following: H1 valve = position B H19 valve = position B H4 (blood pump) = On (ramping to target rate) H12/H13/H20 = Auto Air Trap Level Control (done with call to startAirTrapControl() function). H10 = Off H5 (ejector) = off (retracted)
These overrides need to be updated to Leahi style (returns BOOL, takes message as param, is called by message handler, new Dialin message ID needed for override message, set/reset are merged into single message handling function, etc...).
verify_vitals_pop_up_is_opened_automatically_after_an_interval need to be placed before checking the request in test.verify(waitFor("'vitalsrequest == True'", 1000), "UI Sending the request to FW")
This structure is for message payload. There is nothing to pack in this specific structure, but I think we should be in the habit of packing ALL payload structures in case a future structure change does require packing.
Combine set/reset for the 2 override functions. And use the helper functions in TestSupport.h (like over override handling functions do). f32Override() for the prime volume override and u32BroadcastIntervalOverride() for the broadcast interval override.
We do need to bring blood prime sub-mode into treatment mode (integrate). So signals that blood is primed and it's time to transition from blood prime to dialysis are needed. Treatment mode needs to be updated to include blood prime as a state (the first) in the treatment mode state machine and that state needs to call the exec function in this BloodPrime unit.
Vinayakam Mani What is the advantage of doing that? If I go with Beta_1_9_HW, I will need to rename the flag when we start using Beta 2+. And rename it in all the scripts in different repository when starting with 2.0. While I use a more generic flag, I won't need to do that. So just using a better name I am sparing myself hours of work in the future.
Then the FPGA get function seems mis-named. Pulse width is not really the same as speed. It is the inverse of frequency, which can be converted to speed.
