cogeotiff/rio-tiler-crs

A rio-tiler plugin to create tiles using TMS (WebMercator or others)

rio-tiler-crs's Issues

Archive rio-tiler-crs

we added TMS support in rio-tiler cogeotiff/rio-tiler#285

Missing kwargs override introduced in rio-tiler 2.0b7

https://github.com/cogeotiff/rio-tiler/blob/master/rio_tiler/io/cogeo.py#L277

rio-tiler-crs/rio_tiler_crs/cogeo.py

Line 149 in 12987a2

if isinstance(indexes, int):

checkout https://github.com/ungarj/tilematrix

Bounds should be in WGS84 not in TMS projection

rio-tiler-crs/rio_tiler_crs/tiler.py

Lines 120 to 121 in 1dad4d6

	bounds = transform_bounds(src_dst.crs, tms.crs, *src_dst.bounds, densify_pts=21)
	center = ((bounds[0] + bounds[2]) / 2, (bounds[1] + bounds[3]) / 2, minzoom)

mypy error: "COGReader" gets multiple values for keyword argument "tms"

The new version of COGReader (inherited from rio_tiler.io.cogeo.COGReader) gives a mypy error when used in a function.

This:

def test(asset, tms: morecantile.TileMatrixSet = default_tms):
    with COGReader(asset, tms=tms) as cog:
        return cog.info

will return "COGReader" gets multiple values for keyword argument "tms"

Users need to add # type: ignore to silent this error

def test(asset, tms: morecantile.TileMatrixSet = default_tms):
    with COGReader(asset, tms=tms) as cog:    # type: ignore
        return cog.info

ref: #5 (comment)

switch to GitHub actions

Version 3 pre-release / release master ticket

3.0b.1 - (2020/07/22)

rio-tiler-crs/CHANGES.txt

Line 4 in 4467ca8

- Use rio-tiler 2.0b1 and its new COGReader and STACReader

This is not really a breaking change itself but it requires the new rio-tiler beta which breaks a lot of things so it's safer to use a major version.

In Addition to the switch to rio-tiler.io.COGReader as parent class (we just extent to add tms and new tile method), we've added STACReader (following the same pattern).

create proper repo for `projectile`

Sadly the name projectile is already taken on pypi so we need another name

update to `MissingAssets` from rio-tiler 2.0b4

rio-tiler-crs/rio_tiler_crs/stac.py

Lines 107 to 110 in dc16a68

	if not assets:
	raise InvalidBandName(
	"assets must be passed either via expression or assets options."
	)

Raise rio-tiler exception

rio-tiler-crs/rio_tiler_crs/stac.py

Lines 106 to 109 in 4467ca8

	if not assets:
	raise Exception(
	"assets must be passed either via expression or assets options."
	)

should be
https://github.com/cogeotiff/rio-tiler/blob/master/rio_tiler/io/stac.py#L226-L229

cc @geospatial-jeff