Skip to content
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

[Processing] GRASS7 v.distance (implements Redmine #12817) #2787

Merged
merged 1 commit into from Mar 2, 2016
Merged

[Processing] GRASS7 v.distance (implements Redmine #12817) #2787

merged 1 commit into from Mar 2, 2016

Conversation

ghost
Copy link

@ghost ghost commented Feb 12, 2016

This PR implements GRASS7 v.distance into Processing (implements #12817).

There are two outputs for this algorithm:

  • A copy of the 'from' table with the updated attributes (for cat/dist/to_x/to_y/etc.).
  • The vector with shortest lines between 'from' and 'to' vectors.

The attributes which will be updated by this algorithm should be created in the original 'from' table (they are not created on the fly). As you can update multiple fields at the same time, I haven't use ParameterTableField, which only permit the selection of one field. You have to write the list of fields to update by hand but there is, at least, a control of equality between columns number and 'upload' operations.

There is no direct output (stdout with -p and -a options).

This feature will be for QGIS 2.16.

Thanks to @volaya, @alexbruy , @gioman for reviewing...

@gioman
Copy link
Contributor

gioman commented Feb 14, 2016

thanks for adding this... for incredible as it can seem, we don't had a tool to do distance analysis (not even a plugin as far as I remember)...

alexbruy added a commit that referenced this pull request Mar 2, 2016
[processing] add GRASS7 v.distance (fix #12817)
@alexbruy alexbruy merged commit 61e5799 into qgis:master Mar 2, 2016
@havatv
Copy link
Contributor

havatv commented May 1, 2016

@gioman, the NNJoin plugin was created because v.distance did not work in Processing, so there has been a plugin available.

@gioman
Copy link
Contributor

gioman commented May 1, 2016

@havatv thanks for the heads up!

@ghost ghost deleted the processing_vdistance branch December 18, 2016 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants