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
Raise nicer error message if factor is not integer #14794
Conversation
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
Remote data passed here. Does this mean we no longer need #14789 ? 🤯 |
Perhaps #14789 was just a glitch in the Matrix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, I just suggest including a reminder of the type requirement in the documentation too
Description
With this PR,
discretize_model
will now raise an informativeValueError
ifmode='oversample'
andfactor
does not have an integer value.