search

bjlittle / geovista

Cartographic rendering and mesh analytics powered by PyVista
https://geovista.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
159 stars 25 forks source link

API change for `bridge.Transform.from_points` #1107

Open bjlittle opened 4 weeks ago

bjlittle commented 4 weeks ago

📰 Custom Issue

Align the use of the zlevels kwargs for geovista.bridge.Transform.from_points with the rest of the Transform API i.e., drop support for this being an array of values.

Support optional zs instead. See the point cloud use case.

This change will underpin follow-on work to the Transform API such that it supports volumes rather than surfaces, see #15.

Note, also impacts geovista.common.to_cartesian.

welcome[bot] commented 4 weeks ago

📢 Nice one! Your first issue! Thanks for telling us how to improve geovista 📢