search

GeoTIFF / georaster

Wrapper around Georeferenced Rasters like GeoTIFF and soon JPG and PNG that provides a standard interface
Apache License 2.0
84 stars 32 forks source link

Support for DataView #44

Open qpincon opened 4 years ago

qpincon commented 4 years ago

It would be nice if DataView was supported for parsing, as for now only ArrayBuffer is and a view onto it is not. Right now if multiple data are stored into a single ArrayBuffer the only way for the parser to work is to copy the interesting part into a new ArrayBuffer, copy which could be avoided. Thank you !

DanielJDufour commented 4 years ago

Hi, @pinconquentin . That's a great idea! For GeoTIFFs, georaster depends on geotiff.js, so we will have to add a fromDataView function to geotiff.js. Would you like to post an issue at https://github.com/geotiffjs/geotiff.js/issues? Also, would you like to work on this?

qpincon commented 4 years ago

Hello @DanielJDufour, sorry for the late response. I will probably post an issue there yes. Unfortunatly, I don't have a lot of time in my hands to work on this right now. Do you think there is a lot of work involved ? Thank you for your response anyway.

DanielJDufour commented 4 years ago

I would estimate that there would be about 1-2 days worth of work to add a fromDataView function to geotiff.js and add DataView support to georaster.