rename COGReader to Reader and remove relative import
#534
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
vincentsarago
commented
Oct 13, 2022
| @@ -8,13 +8,18 @@ | |||
| import attr | |||
| import numpy | |||
|
|
|||
| from .errors import ( | |||
| from rio_tiler.errors import ( | |||
There was a problem hiding this comment.
I've changed my mind about absolute/relative import. IMO it's much more cleaner with absolute import
vincentsarago
commented
Oct 13, 2022
| @@ -1,5 +1,8 @@ | |||
| """rio-tiler.io""" | |||
|
|
|||
| from .base import BaseReader, MultiBandReader, MultiBaseReader # noqa | |||
| from .cogeo import COGReader # noqa | |||
| from .rasterio import Reader | |||
There was a problem hiding this comment.
In preparation for merging #530, it makes sense to me to rename cogeo to rasterio
vincentsarago
commented
Oct 13, 2022
| from .stac import STACReader # noqa | ||
|
|
||
| # Keep Compatibility with <4.0 | ||
| COGReader = Reader |
There was a problem hiding this comment.
I believe most user already do from rio_tiler.io import COGReader not from rio_tiler.io.cogeo import COGreader
This comment was marked as outdated.
This comment was marked as outdated.
vincentsarago
changed the title
COGReader to Reader and remove relative import
geospatial-jeff
approved these changes
Oct 13, 2022
CHANGES.md
Outdated
| @@ -8,6 +8,18 @@ | |||
|
|
|||
| * remove python 3.7 support | |||
| * update rasterio requirement to `>=1.3` to allow python 3.10 support | |||
| * rename `rio_tiler.io.cogeo` to `rio_tiler.io.rasterio` | |||
| * rename `COGReader` to `Reader` | |||
There was a problem hiding this comment.
Should mention here this is aliased to COGReader for backwards compatibility.
* remove individual options attributes * add typeDict for options * allow empty options
vincentsarago
added a commit
that referenced
this pull request
Oct 19, 2022
* update rasterio requirement (#517) * update rasterio requirement * remove python 3.7 * update changelog * reader functions returns ImageData object (#515) * reader functions returns ImageData object * update changelog * remove async base reader (#520) * remove gcp reader (#521) Co-authored-by: vincentsarago <vincent.sarago@gmail.com> * avoid warning in tests * generalize buffer option (#519) * generalize buffer option * refactor buffer calculation * better tests * remove min/max zoom from baseclass attribute (#522) * remove min/max zoom from baseclass attribute * fix benchmark * split zooms methods and add tests * rename function * move apply_expression into ImageData class and use `b{ix}` for band names (#523) * move apply_expression into ImageData class and use `b{ix}` for band names * fix test * make sure we have the good number of bands * Apply expression part2 (#525) * ImageData.apply_expression return a new ImageData object * refactor Points method and deprecate asset_expression * remove unused option * Update rio_tiler/models.py * fix regex * add PointData class (#526) * add PointData class * fix test * remove deprecated * fix test 2 * better expression parsing for stac * docstring fixes * only use WarpedVRT when doing reprojection or nodata overwride (#529) * update baseclass to remove useless kwargs * update changelog * forward dataset statistics to ImageClass (#531) * forward dataset statistics to ImageClass * better type and autorescale * one more test * rename `COGReader` to `Reader` and remove relative import (#534) * rename COGReader to Reader and remove relative import * rasterio * debug * merge from branch * add note for backwards compatibility of COGReader * remove individual options attributes (#535) * remove individual options attributes * add typeDict for options * allow empty options * make sure to not change the input array * add ImageReader for non-geo images (#536) * add ImageReader for non-geo images * update changelog * Xarray reader (#530) * sketch out Xarray support * Update rio_tiler/io/xarray.py Co-authored-by: Ryan Abernathey <ryan.abernathey@gmail.com> * read tile * assume crs is in the dataset * upside down * dependencies * cleanup * add more methods and cleanup * Update rio_tiler/io/xarray.py Co-authored-by: Tom Augspurger <tom.augspurger88@gmail.com> * enumerate from 1 * fix * add dataset_statistics and notebook * add XarrayReader in top level import * add tests * update notebook * update changelog Co-authored-by: Ryan Abernathey <ryan.abernathey@gmail.com> Co-authored-by: Tom Augspurger <tom.augspurger88@gmail.com> * add migration docs * update readme * update notebooks * update docs Co-authored-by: Jeff Albrecht <geospatialjeff@gmail.com> Co-authored-by: Ryan Abernathey <ryan.abernathey@gmail.com> Co-authored-by: Tom Augspurger <tom.augspurger88@gmail.com>
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.
ref #532