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

Warn rather than fail if the webcam is not found #488

Open
DominicOram opened this issue Sep 14, 2024 · 1 comment
Open

Warn rather than fail if the webcam is not found #488

DominicOram opened this issue Sep 14, 2024 · 1 comment

Comments

@DominicOram
Copy link
Contributor

Occasionally the webcam can become offline (see https://ispyb.diamond.ac.uk/samples/sid/5853099). Given that the webcam is only really for troubleshooting issues Hyperion should be happy without it so if it is offline we should just log a warning and continue.

Acceptance Criteria

  • Hyperion will log a warning and continue if the webcam is offline
@DominicOram
Copy link
Contributor Author

I have a hotfix on the beamline that does this but it requires more investigation as to why we are failing at all. GDA reads from the same webpage and does not regularly fail to get an image. The only differences between how GDA does it and how we do it are:

  • We use aiohttp, GDA uses urllib
  • We explicitly raise on error codes, GDA does not and will always try and read the data

The hotfix on the beamline now does not check for error codes and carries on anyway, we should keep an eye on it and see if there are cases where we get an error code but still get a valid image. In which case we should just ignore the error code (maybe logging a warning)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo This Sprint
Development

No branches or pull requests

1 participant