Martin Landa
Martin Landa
@seidlmic Can you send me a sample data to reproduce it?
1) Download sample data ``` ./result/bin/grass --tmp-location EPSG:4326 --exec g.download.location https://grass.osgeo.org/sampledata/north_carolina/nc_basic_spm_grass7.tar.gz name=nc_basic path=~/grassdata ``` 2) Enter sample Location in GUI mode ``` ./result/bin/grass --gui ~/grassdata/nc_basic/PERMANENT/ ``` 3) Load `elevation` map...
@HuidaeCho I tested the new `separator` parameter, everything works as expected. I just have little comment on running `v.edit` on a file with different separator than used in input file:...
> > ``` > > head -n2 /tmp/batch > > tool,cats,coords > > break,1,"637108.11963224,257241.783755701" > > > > v.edit map=roadsmajor_upper_half batch=/tmp/batch > > ERROR: Unknown batch column 'tool,cats,coords' in line...
@HuidaeCho Can we manage for 8.4.0 or will be postpone for 8.5? It is a new feature so I am not sure if it's a good idea to introduce it...
> I will not be able to look at and test this this week, but please hold on, I see there are Mac specific changes now that need hands-on testing/review....
> As for the implementation, there is couple of other places which may need fixing. g.extension needs to do something, but I didn't @wenzeslaus I just found this f6ac969: The...
@nilason Currently MacOS CI fails with ``` ERROR: Failed to create configuration directory '/Users/runner/Library/GRASS/8.5' with error: No such file or directory ``` which is triggered by https://github.com/imincik/grass/blob/config-dir-env-variable/lib/init/grass.py#L404 The path (`/Users/runner/Library/GRASS/8.5`)...
> examine what. `lib/gis/home.c` worries me a little bit as it seems to figure out the path as well, at this point duplicating what is in Python which is more...
@wenzeslaus if you have no objections, we can merge this PR. Please can you update your review?