georaster icon indicating copy to clipboard operation
georaster copied to clipboard

Support for DataView

Open qpincon opened this issue 5 years ago • 3 comments

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 !

qpincon avatar Feb 04 '20 15:02 qpincon

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?

DanielJDufour avatar Feb 04 '20 15:02 DanielJDufour

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.

qpincon avatar Feb 05 '20 16:02 qpincon

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.

DanielJDufour avatar Feb 06 '20 04:02 DanielJDufour