Skip to content
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/3.2.0 #497

Merged
merged 141 commits into from
Dec 28, 2022
Merged

Release/3.2.0 #497

merged 141 commits into from
Dec 28, 2022

Conversation

mpu-creare
Copy link
Contributor

@mpu-creare mpu-creare commented Dec 14, 2022

3.2.0 release

Features

  • Added the "none" interpolator, which disables the Interpolation node of Mixin
  • Enabled nearest-neighbor interpolation for N-D stacked coordinates (ENH: Enabling NN interpolation for nD stacked coordinates. #488)
  • Implemented Xarray interpolation for dependent coordinates (multi-dimensional stacked coordinates)
  • Improved caching
    • Extra diemnsions are discarded when caching datasource and compositor outputs
    • Added CRS-agnostic caching
  • Added utility function that generates a JSON spec of available PODPAC nodes. This spec can be used to make UI elements
  • Added the get_source_data method for Datasource nodes to bypass interpolation (Feature: get_source_data #485)
  • Added the get_bounds method to Node to retrieve the coordinate bounds (ENH: Node get_bounds method. #486)
  • Added Node point prober features that will evaluate every node in a pipeline at a point and record its value. Useful for debugging complex pipelines.
  • Added AffineCoorindates (Affine coordinates #491) these are stacked spatial shaped coordinates that are parameterized by a geotransform. Replaces RotatedCoordinates.
  • OGC.WCS Node can now handle multi-band geotiffs

Bugfixes

  • Fixed a number of floating point coordinates discrepancies
  • Rasterio node will no longer attempt to use overviews when no overviews are present.
  • Fixed geotransform shape transpose error
  • Fixed numpy deprecation of using list indices
  • Fixed regressions due to dependency updates

mpu-creare and others added 30 commits April 19, 2021 16:54
Enables users to retrieve raw, uninterpolated data from sources.
* Demonstrate output caching at the top. This is the primary usage of the cache.
* Adds Disk Cache section.
* Adds S3 Cache section.
* Adds Advanced Usage section with caching other objects, cached_property, and cache expiration.
…l before calling super Node eval. This way the Node eval does not need to remove extra dimensions or deal with crs transformation for cache coordinates.
…saves the correct _requested_coordinates for debugging in DataSource and Compsitor eval.
…wrap the super eval (instead of calling self._eval)
Co-authored-by: mpu-creare <mpu-creare@users.noreply.github.com>
Co-authored-by: mpu-creare <mpu-creare@users.noreply.github.com>
Feature: general get_source_data for datasources and tile compositors.
Co-authored-by: mpu-creare <mpu-creare@users.noreply.github.com>
Co-authored-by: mpu-creare <mpu-creare@users.noreply.github.com>
@mpu-creare mpu-creare self-assigned this Dec 14, 2022
@mpu-creare mpu-creare requested a review from jmilloy December 20, 2022 21:51
@jmilloy
Copy link
Collaborator

jmilloy commented Dec 21, 2022

@mpu-creare Is there anything specific you want me to focus on? I'm curious how thorough of a review you are interested in. It's been awhile since I've been in this repo.

@mpu-creare
Copy link
Contributor Author

@jmilloy it's been a while since we've made a release. I just wanted a quick sanity check from another set of eyes. Most of the changes happened months ago and were reviewed as part of other PRs.

Thanks for asking!

Copy link
Collaborator

@jmilloy jmilloy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is a lot stuff that we worked through thoroughly before merging. I found a typo in stacked coordinates that must not come up much by running the tests, and ran the black autoformatting.

@mpu-creare
Copy link
Contributor Author

Thanks a lot @jmilloy:

  • I'm surprised the formatting was that off -- my git hook prevents me from checking in anything without the formatting
  • Thanks a lot for catching that typo -- would have been a huge pain at some point no doubt. I'm also surprised the unit tests did not catch that.

Merging...

@mpu-creare mpu-creare merged commit 13d3d3d into main Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants