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

reload cameras #3

Closed
leoyala opened this issue Aug 14, 2024 · 2 comments · Fixed by #8
Closed

reload cameras #3

leoyala opened this issue Aug 14, 2024 · 2 comments · Fixed by #8
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@leoyala
Copy link
Collaborator

leoyala commented Aug 14, 2024

🚀 Feature Request

At the moment, when a new camera is connected to the system and xilens is already running, the application needs to be closed and re-opened in order for the app to be aware of the new camera. A better approach would be to create a menu option or button to reload the available cameras.

🔈 Motivation

It is too much of a hassle to have to close the app just to connect to a new camera.

🛰 Alternatives

To create a service that periodically checks for new cameras. The disadvantage is that this would be running constantly, while a new camera is only connected sporadically.

@leoyala leoyala added the enhancement New feature or request label Aug 14, 2024
@leoyala leoyala self-assigned this Aug 14, 2024
@leoyala leoyala added this to the v0.2.1 milestone Aug 14, 2024
@leoyala
Copy link
Collaborator Author

leoyala commented Aug 14, 2024

The current behavior is that when a camera already exists in cameraListComboBox, the item is not removed even if the camera is no longer connected, ideally the item should be removed.
Even more, cameras are identified in the combo list by the model number, for example: MQ022HG-IM-SM4x4VIS which creates a problem when two cameras of the same model are connected to the system.

Potential solution 💡 : Identify cameras by model and reference number e.g.: MQ022HG-IM-SM4x4VIS-9.2.4.1

@leoyala
Copy link
Collaborator Author

leoyala commented Aug 14, 2024

The adopted solution is to identify each camera in cameraListComboBox with the pattern: cameraModel@sensorSerialNumber. Such that the camera model can then be queried by splitting the string using the character @.
This needs to be handled carefully because the mapper of available cameras only holds camera model at the moment, so a conflict between a camera item having the handle of a different camera might occur.

@leoyala leoyala mentioned this issue Aug 15, 2024
10 tasks
@leoyala leoyala linked a pull request Aug 15, 2024 that will close this issue
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant