-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
QgsFeature python optimisations #57482
Conversation
@nyalldawson Thanks. Have you been able to test it and measure the improvement? |
With a similar test to the loop from #57470, calling 10.5 million times Before: After: With the optimisations from #57470: Before (#57470 alone): After (this + #57470): So no where near as dramatic as #57470, but still worthwhile! |
Avoid multiple checks for detachment for each attribute
Avoid constructing a bunch of default QVariant objects which we will just replace immediately
@troopa81 good to merge now? |
Optimise some more frequently called methods for use in PyQGIS