name for bounds in wgs84 property in next rio-tiler version

[vincentsarago]

rio-tiler/rio_tiler/io/base.py

Lines 46 to 60 in 1b30e41

	@property
	def geographic_bounds(self) -> BBox:
	"""return bounds in WGS84."""
	try:
	bounds = transform_bounds(
	self.crs, WGS84_CRS, *self.bounds, densify_pts=21,
	)
	except: # noqa
	warnings.warn(
	"Cannot dertermine bounds in WGS84, will default to (-180.0, -90.0, 180.0, 90.0).",
	UserWarning,
	)
	bounds = (-180.0, -90, 180.0, 90)
	return bounds

In next rio-tiler version (currently in dev, branch https://github.com/cogeotiff/rio-tiler/tree/rio-tiler-v3) we have replaced BaseReader.bounds which used to store the dataset bounds in WGS84. Now BaseReader.bounds should return the actual dataset bounds (we have added BaseReader.crs to be able to translate it to other projection).

Having bounds in WGS84 is still useful (e.g for TileJSON) so I have added a property called geographic_bounds. I'm not quite happy with the name (especially because for non-earth dataset it won't work nor make sense to have bounds in wgs84).

I'm open to any feedback.

Other idea:

• we could remove geographic_bounds and add a get_bbox(self, crs: rasterio.crs.CRS = CRS.from_epsg(4326)) method to return dataset bounds in any wanted projection.