Skip to content

Added device IMI Norgren Cadent 3#5

Open
BenKrafted wants to merge 4 commits into
zserg8:developfrom
BenKrafted:cadent3
Open

Added device IMI Norgren Cadent 3#5
BenKrafted wants to merge 4 commits into
zserg8:developfrom
BenKrafted:cadent3

Conversation

@BenKrafted

Copy link
Copy Markdown

Added support for Cadent 3 syringe pump.

Initial commit based on TECAN Cavro
* added device to package init
* added ipynb script to test
* added user manual
* catch 0xFF (decode error) but part of the protocol
@zserg8 zserg8 changed the base branch from master to develop July 11, 2024 16:03
@zserg8

zserg8 commented Jul 11, 2024

Copy link
Copy Markdown
Owner

Sorry for not coming to it earlier. Looks very good, thanks for the effort, I just need to think about a better workaround for that hack in connections.py

@BenKrafted

Copy link
Copy Markdown
Author

Sorry for not coming to it earlier. Looks very good, thanks for the effort, I just need to think about a better workaround for that hack in connections.py

No worries. Indeed the workaround is not ideal. If you keep it open I might have some minor improvements to push.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants