-
Notifications
You must be signed in to change notification settings - Fork 367
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
DOC: What's new for v0.21 release #2074
Conversation
I think this is good to go now, just need to update the release date with whenever we decide to tag a new release. When building the docs, I noticed that we do get some warnings on the SRTM module, but apparently the docs still build. The warning I think is related to trying to document the inherited class attribute, which seems to be a bug upstream: docs/source/reference/generated/cartopy.io.srtm.SRTM1Source.rst:29: WARNING: autosummary: failed to import SRTM1Source.crs These are also present on the CI builds, so we should probably raise warnings to errors on CI to catch these sooner. It looks like we can just ignore these for now, but we should probably try and figure out a better fix or document them without autosummary. |
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.
Just leaving here until we're pencils down.
@dopplershift, @QuLogic do either of you have any preference for when we release or if anything else needs to go in? |
I don't have anything else that needs to go in. #1951 was mentioned, but I'm ok if that gets punted off. |
Shapely v2+ changes the C API access, so pin to earlier versions until this is fixed.
@dopplershift , I think I got all of the locations with shapely versions and pinned those <2. I put the release date for tomorrow September 9th. We can move that around still. |
Preparing for a new v0.21 release by adding a What's New page and documenting what was removed and deprecated in this version.
It doesn't look like we ever settled on moving the docs somewhere else (xref: #1765) , so we will likely have to manually upload them again.
NOTE: Date still needs to be added to the top of the page.