-
Notifications
You must be signed in to change notification settings - Fork 67
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
Pre 0.4.3 release #575
Merged
Merged
Pre 0.4.3 release #575
+5,538
−1,917
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* updated MosaicAnalyser section of quickstart notebook * A few other tweaks * Typo
….1 (#544) Bumps [org.apache.maven.plugins:maven-assembly-plugin](https://github.com/apache/maven-assembly-plugin) from 3.7.0 to 3.7.1. - [Release notes](https://github.com/apache/maven-assembly-plugin/releases) - [Commits](apache/maven-assembly-plugin@maven-assembly-plugin-3.7.0...maven-assembly-plugin-3.7.1) --- updated-dependencies: - dependency-name: org.apache.maven.plugins:maven-assembly-plugin dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Merged with the expansion of the grid
* fixing deletion / typing issues for checkpointing * fix diff * does it build on GH? * changed some of the python tests to allow the package to build with checkpointing by default * updated R tests to allow checkpointing by default * prep to pass around tmp prefix * added RST_Type function to get type for each raster band * added rst_updatetype * added python bindings and docs * added R tests * fix to SQL reg for RST_Transform * Feature/tin mesh (#569) * added methods for triangulation and interpolation, data for testing * updated interpolation function to use an STRTree * fails due to row size exceeding heap * working generator for unconstrained triangulation * conforming triangulation appears to be working * added st_interpolateelevation expression and updated wkb / wkt writers to respect coordinate dimension * removed buffer for input masspoints * first pass at rasterization, added toWKT and toWKB that can explicitly specify coordinate dimensions * added a GDALRasterize operator to write outputs * added RST_DTMFromGeoms initial implementation and tests * fixed issues with interpolation and triangulation split points missing Z values * updated tests to use 1km subset of masspoints * fixed failing tests * ran black on python module * added python bindings for new functions * added triangulation and interpolation python tests * added python test for raster function * added python test for raster function * added SparkR tests for vector functions * added SparkR tests for raster function * added sparklyr tests for vector functions * added raster function tests to sparklyr bindings / fixed typo in sql registration for rst_dtmfromgeoms * description of st_triangulate * mistake in BNG grid * grid points are silently dropped if not in triangulated area (will be set equal to NoData value when raster written) * updated logic of rasterize method to handle empty geometry collections * hardcoded algorithm tolerance to zero * check / fix failing test * small docs updates * minor docs fix, split out tolerance into two params * updated docs / added examples for st_triangulate * added docs for interpolateelevation and dtmfromgeoms * ran isort, reverted h3 version bump, removed unused import
…rst_pixelcount. (#576) Co-authored-by: Michael Johns <[email protected]>
* formatter, added rst_setsrid to docs * standardize on rst_tooverlappingtiles, mark rst_to_overlapping_tiles as deprecated * fixed error in RST_WorldToRasterCoordY (was returning x) * marked rst_to_overlapping_tiles deprecated in MosaicContext * updated RST_PixelCount to take extra params * fixed failing python test * added rst_write expression for writing rasters * fixed R test * added cutline option to RST_Clip * minor docs tweak * `MosaicRasterGDAL` does not need `memsize` at instantiation. Will be estimated from band data types and sizes if not known * formatter run on the python bindings
…pointing disabled, added checkpointing explanation and instructions to documentation
* fixes to triangulation; exposed split point finding algorithm choice * updated R bindings and tests * fixed python bindings and tests * fixed failing test (sidecar files not cleaned up between checkpoint on / checkpoint off tests) * updated docs to reflect new sigs of triangulation funcs
* changes to the RasterAsGridReader and ReTileOnRead processes * removed some cruft * switch runners * fixed failing zarr tests * disabled the test for netcdf with checkpointing disabled
mjohns-databricks
approved these changes
Nov 12, 2024
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.
lgtm
* changes to the RasterAsGridReader and ReTileOnRead processes * removed some cruft * switch runners * fixed failing zarr tests * disabled the test for netcdf with checkpointing disabled * added function `RST_AsFormat()` to change raster format / driver in-situ * fixed zarr test * added python bindings and tests
* updated the logic of some of the statistical raster functions to work with small clipping outputs
This was referenced Nov 21, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For review before we go ahead and merge in to main and start the release process for 0.4.3