-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Field Calculator, slow performance with 2.1.0-104 #17922
Comments
Author Name: joe larson (@oeon) This only occurs when Attribute Table is open.
|
Author Name: Nathan Woodrow (@NathanW2)
|
Author Name: Jürgen Fischer (@jef-n) Fixed in changeset "ccde424dc14ebc82c97e62d3568b1cf15f3bfda9".
|
Author Name: Matthias Kuhn (@m-kuhn) I've noticed a similar latency recently, but in a release build (i.e. without debug output, so Jürgens commits should not make any differece there). Adding a new calculated column to ~1mio rows was incredibly slow. This was with a build from before the recent changes concerning reliable signalling of attribute changes. I hope to get time to looking into batch updating. |
Author Name: dr - (dr -) Also noticed that slow performance occurs only when Attribute Table is open (master, Ubuntu).
|
Author Name: Giovanni Manghi (@gioman) dr - wrote:
confirmed very slow field calculator performance when table is open.
|
Author Name: Matthias Kuhn (@m-kuhn) Is it a regression? |
Author Name: Giovanni Manghi (@gioman) Matthias Kuhn wrote:
it seems faster in qgis 2.0.1 |
Author Name: Matthias Kuhn (@m-kuhn) I could (partially) revert 5c38775 which could probably help here. But this would in turn reintroduce bug #17882 (updates not pushed to the attribute table) http://lists.osgeo.org/pipermail/qgis-developer/2014-January/030018.html |
Author Name: Giovanni Manghi (@gioman) Matthias Kuhn wrote:
personally if I have to choose I would prefer speed (in table edits) rather than #17882 fixed. |
Author Name: Matthias Kuhn (@m-kuhn) I have created a branch which could potentially fix both issues, but I did not have any time at all to test it. If somebody is able to test this and give a feedback... It would be great to have these things solved, but didn't even dare to create a pull request so far. |
Author Name: Matthias Kuhn (@m-kuhn) Fixed in changeset "d378f942a50ad25cc24570e125ea3724e09d79d6".
|
Author Name: Matthias Kuhn (@m-kuhn) My propsed fix was based on wrong assumptions, the main reason for the bad performance was not the fix for #17882 but another change introduced for the relations, which led to tons of unnecessary requests. Fixing that brought back an acceptable speed and the proposed fix implemented this morning gave another remarkable speed boost. Please help testing thiese fixes and report any related bugs or reopen if this issue should still not be solved for any reason.
|
Author Name: Matthias Kuhn (@m-kuhn)
|
Author Name: Salvatore Larosa (@slarosa) Hi Matthias, |
Author Name: Giovanni Manghi (@gioman) Salvatore Larosa wrote:
Hi Matthias and Salvatore, are the fixes already committed in master or to test I have to use Matthias branch? thanks! |
Author Name: Salvatore Larosa (@slarosa) Giovanni Manghi wrote:
Yes, they are in master branch now. |
Author Name: Giovanni Manghi (@gioman) Salvatore Larosa wrote:
very true, when discarding edits it take ages to exit the freeze state. |
Author Name: Jürgen Fischer (@jef-n)
Another user in IRC, Another user in IRC,
|
Author Name: joe larson (@oeon)
Original Redmine Issue: 9315
Affected QGIS version: master
Redmine category:vectors
Assignee: Matthias Kuhn
using OSGeo4W64, 2.1.0-104 while trying to update a field in a large dataset (~300MB) with the Field Calculator ...very slow performance was experienced.
Another user in IRC,
lrssvt
also confirmed experiencing this with this dataset (CARTO1_Lin_6CNivelD.zip) available here #17441-23Related issue(s): #18100 (relates), #18109 (relates)
Redmine related issue(s): 9509, 9519
The text was updated successfully, but these errors were encountered: