Skip to content

Remove incorrect dep install commands for Docker ctr img#218

Merged
ethanjli merged 1 commit intomasterfrom
fix-deps
Jan 26, 2026
Merged

Remove incorrect dep install commands for Docker ctr img#218
ethanjli merged 1 commit intomasterfrom
fix-deps

Conversation

@ethanjli
Copy link
Copy Markdown
Collaborator

The Docker container images are being built in a way that results in different behavior between native Python installations (via UV) and the Docker container. @beniroquai hypothesized that the command removed by this PR is the culprit.

@ethanjli ethanjli marked this pull request as ready for review January 26, 2026 17:36
@ethanjli ethanjli enabled auto-merge January 26, 2026 17:36
@ethanjli ethanjli added this pull request to the merge queue Jan 26, 2026
Merged via the queue into master with commit d4ae04d Jan 26, 2026
17 checks passed
@ethanjli ethanjli deleted the fix-deps branch January 26, 2026 17:57
@ethanjli
Copy link
Copy Markdown
Collaborator Author

@beniroquai FYI the problem is still occurring after this PR; I'm still seeing ERROR [RS232sManager -> MultiManager] Numba needs NumPy 2.3 or less. Got NumPy 2.4. in the logs; and I'm still not seeing a printout of the contents of example_virtual_microscope in the logs.

@beniroquai
Copy link
Copy Markdown
Collaborator

Is this still relevant?

@ethanjli
Copy link
Copy Markdown
Collaborator Author

Yes, this is still a problem. In ghcr.io/openuc2/imswitch:sha-4896268 (from 4896268, running on the base OS image built by openUC2/os-rpi@c373bb9):

  • I am still seeing these "ERROR [RS232sManager -> MultiManager] Numba needs NumPy 2.3 or less. Got NumPy 2.4." messages.
  • The logs include a printout of the contents of imcontrol_options.json (which uses example_virtual_microscope.json as its setupFileName) and I never see a printout of the contents of imcontrol_options.json.
  • When I open ImSwitch's LiveView, the camera stream never loads. Instead, I only see "Loading image..." followed by "Stream nicht aktiv".

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.

2 participants