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
Remove old flux point classes #814
Remove old flux point classes #814
Conversation
🎉 , thanks for the cleanup! I don't remember if I was involved with adding I'll add @joleroi as reviewer now. |
@joleroi Could you please update this notebook to use |
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.
@adonath and I fixed the remaining issues, all looks fine now. Let's see if tests pass ...
4512e2c
to
76037de
Compare
76037de
to
14a396d
Compare
Thanks @joleroi! |
This PR removes the old
DifferentialFluxPoints
andIntegralFluxPoints
classes, from thegammapy.spectrum
module, tests and docs and all uses of it.@joleroi @cdeil There's a
SpectrumFit.compute_fluxpoints()
method which seems to never have worked. I addedraise NotImplementedError
let me know, if I should remove it completely