We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Please describe the use case that requires this feature.
When encountering unsupported (not yet or old, dropped versions) we raise inconsistent, generic python errors.
ValueError for the optics table, OSError else (I think it used to be the more fitting IOError but was changed in an automatic refactoring.
ValueError
OSError
IOError
Describe the solution you'd like
We should use a specific exception in this case that is a ctapipe exception type for this explicit purpose.
Additional context
LST chain wants to add back explicit support for older files, which would be better handled with a specific exception type. cta-observatory/cta-lstchain#1329 (comment)
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Please describe the use case that requires this feature.
When encountering unsupported (not yet or old, dropped versions) we raise inconsistent, generic python errors.
ValueError
for the optics table,OSError
else (I think it used to be the more fittingIOError
but was changed in an automatic refactoring.Describe the solution you'd like
We should use a specific exception in this case that is a ctapipe exception type for this explicit purpose.
Additional context
LST chain wants to add back explicit support for older files, which would be better handled with a specific exception type.
cta-observatory/cta-lstchain#1329 (comment)
The text was updated successfully, but these errors were encountered: