Skip to content
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

Add OPT use case #201

Open
wants to merge 145 commits into
base: master
Choose a base branch
from
Open

Add OPT use case #201

wants to merge 145 commits into from

Conversation

palec87
Copy link

@palec87 palec87 commented May 14, 2024

Main contributions:

  • OPT controller: imswitch\imcontrol\controller\controllers\ScanControllerOpt.py
  • OPT widget: imswitch\imcontrol\view\widgets\ScanWidgetOpt.py
  • Alignment controller: imswitch\imcontrol\controller\controllers\AlignOptController.py
  • Alignment widget: imswitch\imcontrol\view\widgets\AlignOptWidget.py

Other notables

  • added ic4 TIS camera support, imagingcontrol4
  • added arduino support with telemetrix library
  • proposed ScanExecutionMonitor class in ScanControllerOpt.py

Tested with NEMA17 4phase stepper, DMK252 TIS camera
OS: Win

Other tests under way with Hamamatsu ORca.

palec87 and others added 30 commits December 5, 2023 15:14
CommunicationChannel integration for dynamic OPT scan control
@jacopoabramo
Copy link
Collaborator

Build is failing due to this:

ERROR: Ignored the following versions that require a different python version: 0.4.2 Requires-Python >=3.9,<3.12; 0.4.3 Requires-Python >=3.9,<3.12; 0.4.4 Requires-Python >=3.9,<3.13
ERROR: Could not find a version that satisfies the requirement imagingcontrol4>=1.0.1.2418 (from imswitch) (from versions: none)
ERROR: No matching distribution found for imagingcontrol4>=1.0.1.2418

It might be due to the fact that this package is not available on Python version 3.8 and 3.9. @palec87 could you double check if this is the case?

@palec87
Copy link
Author

palec87 commented May 14, 2024

Hi @jacopoabramo, I pushed fixes, ic4 library is only available for windows, so I removed it from setup.cfg file.

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.

None yet

2 participants