[Processing] Add GRASS7 r.reclass inline rules (implements Redmine #5583) #2782
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
|
@medspx many thanks for this, a very important improvement. |
|
Oops, I didn't take time to make tests under MS-Windows and there is a problem with CR characters (with the MS-Windows shell). |
|
Well, I've found the workaround: create a temp file with reclass rules and use it as input for r.reclass command. Tested on GNU/Linux and MS-Windows. |
|
I really think that regardless of being a new tool (so a new feature) we should commit this and have this very important improvement directly into 2.16. After all I don't see how this Processing improvements can be harmful. The same applies to #2781 and #2788 and #2772 and #2747 and #2787 @volaya @alexbruy @jef-n |
alexbruy
added a commit
that referenced
this pull request
Mar 2, 2016
[processing] add GRASS7 r.reclass inline rules (fix #5583)
ghost
deleted the
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.
With this PR, you can write reclass rules in the Processing dialog box (as required in feature request #5583):
This will be for QGIS 2.16 because it is a new feature.
Thanks to @volaya, @alexbruy for reviewing...