-
Notifications
You must be signed in to change notification settings - Fork 3
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
Use DN instead of counts everywhere #338
Conversation
Consider this a breaking change? We can do a 0.9 release for it. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #338 +/- ##
===========================================
- Coverage 86.84% 69.72% -17.13%
===========================================
Files 34 34
Lines 1209 1209
===========================================
- Hits 1050 843 -207
- Misses 159 366 +207 ☔ View full report in Codecov by Sentry. |
Yes, I would say so |
I think a breaking changelog and its good to go. |
The docs build is failing,
and I've not worked out why yet. |
Just add it to the nickpicky list. |
Why do I always forget this… |
We will pin the new version to sunpy 6.0 :P |
In line with sunpy/sunpy#7585, this replaces the usage of
ct/count
withDN
everywhere. This may cause some friction in terms of compatibility with sunpy as prior to v6, the units of any AIA map are converted toct
rather thanDN
. I'm not sure the best way to resolve that. We could of course just pin to >= v6, but that seems aggressive.