Open seichhorn opened 4 years ago
These can be merged as long as they don’t break the functionality of the peristaltic pump. I don’t have a peristaltic pump available to test.
I'll test this afternoon
Let me also pull this branch and test it on my systems before we merge it. I should be able to run this test tomorrow.
This worked fine for my peristaltic pump, there's a minor issue where it wasn't able to parse the default_config.xml file that it attemps loads upon open, I'm trying to trace down why that is and resolve it, but it parsed the protocol/valve config file I use without issue/changes and ran through the protocols properly.
@emanuega @jeffmoffitt Is this okay to merge?
I'd like to get back onto master, can these be merged or are additional changes needed?