Markus Metz

> As far as I understand libdeflate is not a "replacement" to zlib, meaning this is not only a question of configuration. See example from [GDAL](https://gdal.org/development/building_from_source.html#deflate) : [OSGeo/gdal@01b3560](https://github.com/OSGeo/gdal/commit/01b3560c90b3a7ae005dd67c69eae85262adcec3) Yes, support...

When a vector is opened with `Vect_open_update2()`, the topo, cidx, and sidx files are deleted and need to be created anew which happens with `Vect_close()`. That means with a crash...

With [8ef73e8](https://github.com/OSGeo/grass/pull/3459/commits/8ef73e8a9b83bb659d6fe732a45456c02bc44e67) there is a better solution where a map opened in update mode survives with level 2, any changes would be discarded, i.e. if `Vect_close()` can not be called...

I am not convinced with the current proposed solution because in case of a crash this can lead to a modified coor file (modified geometries) and old topo, sidx, cidx...

@marisn This does not solve the problem of a crash. I prefer solution 1 where the support files are deleted when opening and always written out anew when closing. Writing...

There is a strange error in the tests with `Running ./python/grass/pygrass/vector/testsuite/test_pygrass_vector_doctests.py...` resulting in `ERROR: Category index is not up to date` I guess that before the test vector is closed,...

> [...] Will it be slow to rebuild it if there are a lot of features? If topology is already built and you just want that the cidx is marked...

In pygrass, here https://github.com/OSGeo/grass/blob/main/python/grass/pygrass/vector/abstract.py#L454-L470 and here https://github.com/OSGeo/grass/blob/main/python/grass/pygrass/vector/abstract.py#L454 the vector is first closed, then topology is built, then the vector is closed again. This seems wrong to me, topology should be...

This PR raises the question if r.import should support all options and flags of r.in.gdal + r.proj. Please remove calls to `print()` or replace them with GRASS debug messages and...

In the meantime, `r.import` has changed, conflicts need to be resolved. IMHO, this PR should be merged first, then we can think of special cases like Sentinel-2 where e.g. some...