-
Couldn't load subscription status.
- Fork 35
Made ORSTools ready for qgis_process #325
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…s a integer parameter as string)
…se with qgis_process in an headless environment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey,
thanks a lot for the contribution!
I had a quick look at everything but the new processing algorithm - will have to look at that one in detail :)
|
Thanks for the feedback, removed old stuff. |
…and hopefully test for 3.16 runs...).
…r/orstools-qgis-plugin into qgis_process_ready
…r/orstools-qgis-plugin into qgis_process_ready
…rnal plugins (especially headless))
Added sink commitment following qgis-pull-request #59241
Added sink commitment following qgis-pull-request #59241
This reverts commit 89ad098.
|
Hi, this would be a really cool feature to have I think. I would however be -1 on introducing three new processing algorithms for this sole reason. @koebi proposed to hide the new processing algorithms from the toolbox. Could you have a look at how that would work? Otherwise I'd also be open for a video call to discuss details if you would like. |
|
Though currently it isn't beautiful that there are (somewhat) double structures to modify config. Maybe modify gui to use the processing tool internally, too. To safe code lines. Edit: A, now I understand what you are saying. I'll check how to useHideFromToolbox-flag... |
|
So, I have hidden the algorithms now. Though should be documented somewhere that they exist? |
|
@koebi Anything to be done before merged? |
|
Hey,
Do you have any idea why the tests do not run completely? |
|
I have absolutely no clue. And if I initiate them locally I almost instantly hit my apikey-limit. So sadly I am no help here :/ . |
Because I need to run our plugin OS-WALK-EU (which depends on isocrone calculations from ORStools) from within a qgis_process setup headless I needed some upstream changes:
Now I can run something like this from the command line:
followed by:
Due to
None/null-Values and[]as parameters, only the json-piping is working.