-
-
Notifications
You must be signed in to change notification settings - Fork 65
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
Release 0.1 #75
Comments
I'll see if I can improve the test coverage a little. |
Is #59 a real bug? Or just a misunderstanding or something that has been fixed in the meantime? |
I'm playing with the examples from the docs ... everything works so far. The example in http://reproject.readthedocs.org/en/latest/healpix.html gives these warnings:
It's not important, but probably these should be fixed or suppressed? |
Here it should be |
There's also this warning which should be suppressed because it occurs frequently in normal usage (I think):
|
#59 is a real bug The warnings due to the NaN values are annoying so this should fix it: https://github.com/astrofrog/reproject/pull/78 The |
I fixed the other warning you mentioned in #78 too. |
One thing that could be useful for users: say which scipy and healpy version is needed for reproject here: |
For the healpix example from the docs I still get these warnings from wcsaxes. Possible / worth fixing or suppressing?
|
Saving the HEALPIX file I get Astropy deprecation warnings with healpy version 1.8.4.
|
@astrofrog – I tried all examples from the docs again. All work, except for #76 for which you said it's an Astropy / CFITSIO / Macports issue. So 👍 to release now ... thank you !!!! |
OK, I'll file issues about the warnings from wcsaxes and healpy with as short as possible test cases. |
I've made separate issues for the warnings mentioned above at astrofrog/wcsaxes#159 and healpy/healpy#253. |
I think we're good to go for 0.1, since the notebook issue is not something that we can easily control |
Version 0.1 has been released! @cdeil - thanks for all your help! |
I think the stable branch is pointing to bc2cdc2 but the 0.1 tag is pointing to ee8adde , which leads to readthedocs showing a dev version when I select "stable". Not a big deal ... probably it's OK if this is resolved with the 0.2 release .... |
@cdeil - this is because I made a few small changes to the docs after the release (including install instructions). A little annoying esthetically, but will make sure that's not needed next time. |
Aside from #61 and #69, I'm not sure if anything else needs to be done before an initial release. Many of the other issues are not critical for 0.1 and I'd like to have this package see some real-world testing. I'm planning on having frequent releases.
The docs are now working and plots are showing: http://reproject.readthedocs.org/en/latest/
@cdeil - is there anything that you think should be done before a first release?
The text was updated successfully, but these errors were encountered: