dartclub / turf_dart

A turf.js-like geospatial analysis library working with GeoJSON, written in pure Dart.
https://pub.dev/packages/turf
MIT License
63 stars 29 forks source link

Port boolean* functions and tests #88

Closed armantorkzaban closed 1 year ago

armantorkzaban commented 2 years ago

implement peer dependencies:

lukas-h commented 2 years ago

Inspiration: https://github.com/mfogel/polygon-clipping

armantorkzaban commented 2 years ago

@lukas-h, what do you think about implementing 'properties' for FeatureCollection class? During working with unit tests I saw a few occasions where it was used in the geojson files for featureCollections, and I think it could be in some cases beneficial.

lukas-h commented 2 years ago

@armantorkzaban Are you referring to a properties member for FeatureCollection? I would try to avoid that, since that is not part of the GeoJSON-RFC. Can you find a way to keep it separate? Maybe in the future we can solve this with #44 (long overdue...). But for the time being, it would be great to avoid it.

armantorkzaban commented 2 years ago

ATM, when I encounter it in a unit test from TurfJS I take it from the decoded JSON and use it for the rest of the operations. If it is a thing that only TurfJS people use, we can avoid it for now. Will check #44.

lukas-h commented 2 years ago

Yes, let's avoid it for now, but in the future, #44 will be implemented.

armantorkzaban commented 2 years ago

We need a reminder system for such things. Any idea?

lukas-h commented 2 years ago

GitHub projects, I started one, but didn't have the time to complete it. We will have a backlog and sprints in separate boards there.

lukas-h commented 2 years ago

https://github.com/orgs/dartclub/projects/1

armantorkzaban commented 1 year ago

After the discovery another blocking dependencies (rbush) we decided to separated the dependant libraries from this PR and work on them separately. Therefore the task list is shortened and PR will be marked as ready for review.

armantorkzaban commented 1 year ago

the issue definition is updated.