perf: default to ~1km geometry simplification

transition-zero / tz-client

A client repository for accessing FEO data programmatically.

https://docs.feo.transitionzero.org

Apache License 2.0

21 stars 4 forks source link

Closed a-cass closed 10 months ago

a-cass commented 10 months ago

Performance: Default to applying a 1km simplification of geometries.

[x] Dependencies install correctly in a clean environment and code executes;
[x] Test coverage extended; created tests fail without the change (if possible);
[x] All tests passing;
[x] Commits follow a type convention (e.g. https://gist.github.com/brianclements/841ea7bffdb01346392c#type);
[x] Extended the README, documentation and/or docstrings, if necessary;

vercel[bot] commented 10 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
feo-client	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 28, 2023 4:41pm

a-cass commented 10 months ago

Do e.g. power plants have a geometry? If so this will presumably cause oversimplification.

Currently no. But also wary of oversimplification. Going with increasing the client timeout to resolve this issues instead. See #73