Comment Search

remove and add delay i waitForObjectExists if needed

I am marking it as resolved since it is temporary driver

marking it as resolved since it is temporary driver

Remove extra blank line.

It is a little bit concerning:
Could you please test/describe the following scenarios:

• What if we have more than one Heparin type supported?
• What if none (incorrectly) has been defined in the configuration (do we get an index out of bounds)?

Done

Done

add visible only in service

updated

increasing it to 50 would cause instability in our testing. 25 seems to keep it stable at the moment

Updated.

This is a lot of memory for an average with little value. Assume we want moving 3-sec average and samples are added in priority task every time (so 100/sec). Unlikely that we will have real samples at 100 Hz. Maybe 10 Hz. So recommend adding a new sample to the running average every 100 ms (every 10th time in priority task) so we can reduce the number of samples in the average from 300 to 30 to be more reasonable and save memory.

The steps have been modified as per suggestion

Do we need a Dialin Tx param set message and message handling function (like HD in Denali had)?

Report has been updated in the jira

Add space between if and (.

Now using setText(obj, text) method and moved to configuration folder utility.py file

As per previous test script removed this function and using set_property_text method

Remove extra blank line.

Not a comment to be done right now.
But better to create arrays of test data on top of the function/file, loop through them, and call the function (verify_td_and_dd_versions) to use those.
So we can more easily add multiple test data cases later to cover more edge cases.

moved to shared file and update to have the object as a parameter

Be consistent on whether you have a blank line between break and next case.

Renamed the function setObjectText

Done

rename to set_property_text()