-
Notifications
You must be signed in to change notification settings - Fork 126
Implement flow rate for Tecan EVO backend #549
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement flow rate for Tecan EVO backend #549
Conversation
|
This change works for me. |
|
@nedru004 does it have the correct flow rate? on the forum we discussed perhaps the units are wrong would you be able to confirm this? then we can fix and/or merge this https://discuss.pylabrobot.org/t/subdevice-error-in-evo200-dispensing/166 |
|
The multiplier in the comments with the question mark seems to give the correct steps per uL. I changed to those values and checked the volume dispensed with a pipette which was close to the volume requested. I am running the liquid system with air right now. I assume I will have error in the volumes since so much air is compressed between the pipette and the diluters, but hopefully not much. |
nice
in the tubes that connect the pumps to the channel heads? |
Yeah. I dont have the air LiHa and I dont have the wash/flush protocol, so I am just running the syringes/pumps dry at the moment. I am not sure if that is bad for the machine (lubrication?) and if that has an effect on volume at fast flow rates. That is why I slowed it down. |
|
I do not know for sure but I think it would change the liquid handling characteristics. Do you want to make a PR with your changes into this branch? Then we can have a verified implementation. Don't worry about fixing the unit tests, I can do that if you want. |
|
updated the tests to reflect the new behavior @nedru004 can you please confirm this last version is working as expected for you? then i will merge this branch |
|
This version works for me. Thanks |
|
thanks for confirming! |
|
and congrats on your first commit being merged into plr:main! :D |
Summary
Testing
pytest -q(fails: ModuleNotFoundError: No module named 'serial', 'responses', 'websockets', 'werkzeug', 'requests')https://chatgpt.com/codex/tasks/task_e_6840fd261fc0832b880c36b96367ad43