JuliaEarth / GeoStats.jl

An extensible framework for geospatial data science and geostatistical modeling fully written in Julia
https://juliaearth.github.io/GeoStatsDocs/stable/
MIT License
506 stars 60 forks source link

Implement `setindex!` for geospatial data #282

Open juliohm opened 1 year ago

juliohm commented 1 year ago

Currently our geospatial data types implement getindex to access the underlying table of attributes and geometries. We need the same high-level interface with setindex! to save users from having to learn and use Tables.jl explicitly.

davibarreira commented 1 week ago

The getindex seems to be implemented at GeoTables.jl. Is the setindex! also to be implemented there?

juliohm commented 1 week ago

That was the original idea, but we need to wait on Tables.jl as it currently doesn't provide a setindex API.

We also concluded that the geometry column needs to be immutable for type stability in downstream algorithms.