a small note, the added code could be probably moved to new a fn eg.
clean_db(), Ma
Yes, ideally to grass.script.setup.
But this is only initializing a GRASS session, without any mechanism to terminate a session. Basic usage in the documentation:
… use GRASS modules here
remove the session’s gisrc file to end the session
os.remove(gisrc)
maybe a new fn to terminate a session is needed? This fn could clean temp files like clean_temp() in grass.py and also clean any default sqlite db when terminating the grass session.
a small note, the added code could be probably moved to new a fn eg.
clean_db(), Ma
Yes, ideally to grass.script.setup.
But this is only initializing a GRASS session, without any mechanism to terminate a session. Basic usage in the documentation:
… use GRASS modules here
remove the session’s gisrc file to end the session
os.remove(gisrc)
maybe a new fn to terminate a session is needed? This fn could clean temp files like clean_temp() in grass.py and also clean any default sqlite db when terminating the grass session.
That’s all correct. There would have to be more to make it the same procedure as in grass.py, but eventually we need to do that.
What I meant to add it to grass.script.setup module where we should eventually put all the functions needed to setup a proper GRASS session (or in some other module, but setup seems to be something good enough for now and moving things around while keeping the API is easy in Python).
a small note, the added code could be probably moved to new a fn eg.
clean_db(), Ma
Yes, ideally to grass.script.setup.
But this is only initializing a GRASS session, without any mechanism to terminate a session. Basic usage in the documentation:
… use GRASS modules here
remove the session’s gisrc file to end the session
os.remove(gisrc)
maybe a new fn to terminate a session is needed? This fn could clean temp files like clean_temp() in grass.py and also clean any default sqlite db when terminating the grass session.
That’s all correct. There would have to be more to make it the same procedure as in grass.py, but eventually we need to do that.
What I meant to add it to grass.script.setup module where we should eventually put all the functions needed to setup a proper GRASS session (or in some other module, but setup seems to be something good enough for now and moving things around while keeping the API is easy in Python).
ok, the db cleanup is now (r73704) a function in grass.script.setup
That’s all correct. There would have to be more to make it the same procedure as in grass.py, but eventually we need to do that.
What I meant to add it to grass.script.setup module where we should eventually put all the functions needed to setup a proper GRASS session (or in some other module, but setup seems to be something good enough for now and moving things around while keeping the API is easy in Python).
ok, the db cleanup is now (r73704) a function in grass.script.setup
Great! Thanks. I tried to clean grass.py as much as possible, so I hope it’s possible to make changes there.
That’s all correct. There would have to be more to make it the same procedure as in grass.py, but eventually we need to do that.
What I meant to add it to grass.script.setup module where we should eventually put all the functions needed to setup a proper GRASS session (or in some other module, but setup seems to be something good enough for now and moving things around while keeping the API is easy in Python).
ok, the db cleanup is now (r73704) a function in grass.script.setup
Great! Thanks. I tried to clean grass.py as much as possible, so I hope it’s possible to make changes there.
Unfortunately, just running:
$ grass77 --tmp-location XY --exec g.region -p
…
Execution of <g.region -p> finished.
Traceback (most recent call last):
File “./bin.x86_64-pc-linux-gnu/grass77”, line 2260, in
main()
File “./bin.x86_64-pc-linux-gnu/grass77”, line 2208, in main
clean_all(params.tmp_location)
TypeError: clean_all() takes no arguments (1 given)
That’s all correct. There would have to be more to make it the same procedure as in grass.py, but eventually we need to do that.
What I meant to add it to grass.script.setup module where we should eventually put all the functions needed to setup a proper GRASS session (or in some other module, but setup seems to be something good enough for now and moving things around while keeping the API is easy in Python).
ok, the db cleanup is now (r73704) a function in grass.script.setup
Great! Thanks. I tried to clean grass.py as much as possible, so I hope it’s possible to make changes there.
Unfortunately, just running:
$ grass77 --tmp-location XY --exec g.region -p
…
Execution of <g.region -p> finished.
Traceback (most recent call last):
File “./bin.x86_64-pc-linux-gnu/grass77”, line 2260, in
main()
File “./bin.x86_64-pc-linux-gnu/grass77”, line 2208, in main
clean_all(params.tmp_location)
TypeError: clean_all() takes no arguments (1 given)