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] add interpolation tools #3592

Merged
merged 4 commits into from
Oct 20, 2016
Merged

[processing] add interpolation tools #3592

merged 4 commits into from
Oct 20, 2016

Conversation

alexbruy
Copy link
Contributor

This adds interpolation tools from Interpolation plugin to Processing toolbox. Tests included.

@gioman
Copy link
Contributor

gioman commented Oct 11, 2016

@alexbruy @volaya Alex, many (many) thanks for this contribution. A step forward in removing (bad) redundancy in QGIS and having all the analytical tools in the right place, Processing.

@nyalldawson
Copy link
Collaborator

Really great stuff!

Have you thought about removing the old interpolation plugin too? Is there any reason to keep that around now?

@alexbruy
Copy link
Contributor Author

alexbruy commented Oct 12, 2016

Yes, I was thinking about dropping Raster terrain analysis, Interpolation and Zonal Statistics plugins (maybe also Road Graph) after moving their functionality into Processing.

But there are some functionality, which is not available in Processing, e.g. support for multiple pairs layer/field in interpolation etc. I will try to implement it too. Also need to solve issues with test

@nyalldawson
Copy link
Collaborator

Great! I'd love to see all these c++ plugins go. And this is the perfect approach! (leave c++ in analysis library, front end via processing)

@alexbruy alexbruy merged commit ea0ad5d into qgis:master Oct 20, 2016
@alexbruy alexbruy deleted the processing-interpolation branch October 20, 2016 08:31
@qgib qgib mentioned this pull request May 24, 2019
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