nismod / irv-frontend

IRV frontend
MIT License
1 stars 0 forks source link

Drop mapbox dependency #15

Closed tomalrussell closed 1 year ago

tomalrussell commented 3 years ago

Consider moving basemap to Deck.GL raster bitmap layer, drop react-map-gl and mapbox.

Downside - lose potential to have a vector basemap layer. Need to find an open satellite layer.

mz8i commented 2 years ago

I've been looking at free satellite imagery alternatives - seems a bit hard to find. Some options:

tomalrussell commented 2 years ago

Thanks for looking into this - there is a Sentinel-2 "cloudless" product from EOX

https://s2maps.eu/ > Click "Explore" then "Download" icon on the left

Option 2 Sentinel-2 cloudless WMTS/WMS Access (forever free)

Alternatively, feel free to download the data as rendered image mosaics directly from the provided service endpoints (WMTS or WMS). You may also use these services directly in your application. Just make sure to follow the license conditions like attribution requirements 2016, 2018, 2019, & 2020 and non-commercial usage (2018, 2019, & 2020). The maximum request size is limited to 4096px, but you may stitch multiple requests to fit your needs.

If you are looking for the original GeoTIFF files from 2016 instead, have a look at this blog post. You may download the data as source GeoTIFFs from the AWS S3 bucket eox-s2maps in the eu-central-1 region. Warning: AWS may charge you for download bandwidth. Attribution & License

Please follow the attribution requirements. 2018, 2019, and 2020 - Creative Commons Attribution-NonCommercial-ShareAlike 4.0 International License 2016 - Creative Commons Attribution 4.0 International License

tomalrussell commented 2 years ago

Are we only using mapbox-gl for the CSS now? Or is it still required as a peer/transitive dependency?

mz8i commented 2 years ago

Also, react-map-gl relies on mapbox code for the controls such as ScaleControl etc. The solution would be to replace mapbox with maplibre but I got stuck with that last time I tried (would have to resurrect the branch to see what the main problem was).

tomalrussell commented 1 year ago

@mz8i is this closed as of #28?

mz8i commented 1 year ago

Yes, although we're not dropping the dependency on w basemap library - quite the opposite, in fact - but the API token question is solved