Tim Welch

Results 51 comments of Tim Welch

I've been piecing together the status of this for my own awareness and thought I'd give an update. `turf-boolean-valid` is available as an individual package (https://www.npmjs.com/package/@turf/boolean-valid) for folks to use,...

The example in the Turf docs seems to have the same offset issue (see screenshot). I can at least confirm what you're seeing and would guess it's a bug based...

Actually @pietervdvn this PR was not merged back to turf master, I did the opposite and merged latest turf master to this PR to update it and run the CI...

I think this is an issue with the upstream polygon-clipping library, which was probably swapped in as the implementation for turf.union between the two versions you are using. See this...

Thanks for asking. I encourage you to look at the source, it's quite readable. `boolean-within` checks that each point in polygon 1 is within polygon 2. That is quite fast....

@mfedderly There's a lot of casting happening in these additions. Is your intention that they should be accepted for this PR? So that it doesn't get too big chasing it...

In the meantime, or as a precursor, here's a technique I'm playing with to use `georaster` directly to fetch a smaller window from a larger COG based on the bbox...

Did you try importing geotiff using a dynamic import as the error suggests? Not sure if this handles converting all underlying dependencies too. Something like the following you might be...

@DanielJDufour @MichaelBuhler here's some quick tests in node16 and ts-node that seem to work. Note the jsFail script that reproduces the error, and index.js is successful - https://github.com/twelch/geotiff-dynamic-import-test For ts-node...

@DanielJDufour I think this is ready for (initial) review when you have time. I've tested with maybe 1/3 of the methods using just a single float32 raster. I'm curious if...