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

Misc processing improvements #2471

Merged
merged 2 commits into from Nov 17, 2015
Merged

Conversation

nyalldawson
Copy link
Collaborator

Use QgsStatisticalSummary for statistic calculations:

Switches BasicStatisticsNumbers and StatisticsByCategories to use the QgsStatisticalSummary class for statistic calculation. This removes duplicate code and helps standardise these types of calculations in the one place for easier testing. Additionally, it brings the benefit of some new stats (First/third quartile, IQR, majority and minority) to BasicStatisticsNumbers.

Use faster expression request for ExtractByAttribute:

Like the earlier changes to SelectByAttribute, this switches the algorithm to use an expression feature request. For providers which support compilation (postgres, soon to be OGR) this results in a much much faster extract.

For BasicStatisticssNumbers and StatisticsByCategories algorithms.
Adds extra stats (minority, majority, 1st/3rd quartile and IQR) while
avoiding custom statistics calculation code.
alexbruy added a commit that referenced this pull request Nov 17, 2015
[processing] misc improvements
 - use QgsStatisticalSummary for statistic calculations
 -  faster expression request for ExtractByAttribute alg
@alexbruy alexbruy merged commit f184ec9 into qgis:master Nov 17, 2015
@nyalldawson nyalldawson deleted the processing_fixes branch November 18, 2015 19:35
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.

None yet

2 participants