Update processing instructions #7140
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Previous instructions were not working. Need to `initialize()` Processing before calling for `algorithms()`
The `from qgis import processing` does work in the Python Console from QGIS GUI app, but if running a script for a custom application, the `from processing.core.Processing import Processing` must be called.
DelazJ
reviewed
Dec 24, 2021
There was a problem hiding this comment.
Sorry for the quite late review.
I have some formatting suggestions.
Issue I have with this change is that the chapter is about the Python console inside QGIS, so not one you'd be using from a custom application (am I right that the one you are using here is not the QGIS embedded one?). That said, browsing the docs i can't see any other place your changes will fit in, so we can circumvent my concern by either:
- adding it as a note
- if you confirm that the rest of the instructions are applicable in a custom application context, we can remove the few sentences related to QGIS console and make it a general chapter on python console
| @@ -41,6 +41,12 @@ with the following line: | |||
|
|
|||
| >>> from qgis import processing | |||
|
|
|||
| And if you are running a custom application, this is the import. Also, need to `initialize()` Processing. | |||
There was a problem hiding this comment.
missing word?
Suggested change
| And if you are running a custom application, this is the import. Also, need to `initialize()` Processing. | |
| And if you are running a custom application, this is the import. Also, you need to | |
| :meth:`initialize() <qgis.core.QgsProcessingAlgorithm.initialize>` Processing. |
| @@ -41,6 +41,12 @@ with the following line: | |||
|
|
|||
| >>> from qgis import processing | |||
|
|
|||
| And if you are running a custom application, this is the import. Also, need to `initialize()` Processing. | |||
|
|
|||
There was a problem hiding this comment.
Suggested change
| :: | |
to indicate that the indentation is within a code block, otherwise you get an ugly grey vertical bar on the left
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Previous instructions were not working. Need to
initialize()Processing before calling foralgorithms()