Bug report #16898

Geometry checker crashes when checking a specific vector layer (attached)

Added by Antonio Viscomi almost 7 years ago. Updated over 6 years ago.

Status:Closed
Priority:High
Assignee:Sandro Mani
Category:C++ plugins/Geometry Checker
Affected QGIS version:2.18.11 Regression?:Yes
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:fixed/implemented
Crashes QGIS or corrupts data:No Copied to github as #:24797

Description

hello,
I tried to Use Geometry Checker in a polygon shapefile,

I must check for: -duplicates;
-small overlaps (smaller than 100 sqr. m);
-small gaps (smaller than 1000 sqr. m).

I set create a new layer (with name an path)
then QGIS crashes or freezes.
I try to re-run QGIS,
I reset all option cited above and QGIS crashes yet.
So I think there is a bug that causes crash.
I attach a screenshot and the file I tried to validate
best regards

Antonio

Crash.png (353 KB) Antonio Viscomi, 2017-07-20 06:16 PM

Geologica.zip (147 KB) Antonio Viscomi, 2017-07-20 06:21 PM

bug_record.txt Magnifier (2.75 KB) Antonio Viscomi, 2017-08-04 01:55 PM

Bug screenshot.png (277 KB) Antonio Viscomi, 2017-08-04 01:57 PM

History

#1 Updated by Giovanni Manghi almost 7 years ago

  • Subject changed from Geometry checker to Geometry checker crashes when checking a specific vector layer (attached)
  • Category changed from Geometry to C++ plugins/Geometry Checker
  • Assignee set to Sandro Mani
  • Regression? changed from No to Yes

Instant crash also on Linux.

#2 Updated by Giovanni Manghi almost 7 years ago

No crash on 2.14.16 but also not sure it works as expected, in fact the "fixed" layer is added to the canvas but the result tab is not opened and stay disabled.

#3 Updated by Antonio Viscomi almost 7 years ago

Sorry,
I forget to select OS,
I use windows 7 Ultimate x64
Regards

#4 Updated by Giovanni Manghi almost 7 years ago

  • Affected QGIS version changed from 2.18.10 to 2.18.11

#5 Updated by Antonio Viscomi almost 7 years ago

Also on QGIS 2.99 a48c146 Geometry Checker has a bug:
-I try to check geometry of the same shapefile with the same settings quoted above;
-the check come out fine;
-then I tried to automatically fix errors;
-the progress bar reaches 100%;
-QGIS crashes both in 'Modify input layer' and 'Create a new layer' mode
Best Regards

#6 Updated by Antonio Viscomi almost 7 years ago

Today I've updated QGIS MASTER to build 3c58599,
I tried to check geometry of the same file attached above, with the same settings then,
after checking, I flagged Geometry and the plugin crashed instantly and comes out with the attacched report

#7 Updated by Sandro Mani almost 7 years ago

The geometry checker is pretty broken in master, I'm working to fix it and improve it with multi-layer support, but the work is not yet ready to be merged.

#8 Updated by Antonio Viscomi almost 7 years ago

Sandro Mani wrote:

The geometry checker is pretty broken in master, I'm working to fix it and improve it with multi-layer support, but the work is not yet ready to be merged.

Ok,
many thanks for this information

Good job then

#9 Updated by Sandro Mani over 6 years ago

I've finally got to submitting the PR for the new enhanced geometry checker (https://github.com/qgis/QGIS/pull/5410) for QGIS 3.0+. Testing feedback is very welcome.

#10 Updated by Giovanni Manghi over 6 years ago

  • Status changed from Open to Feedback

Please test again after the patch has been merged in master.

#11 Updated by Antonio Viscomi over 6 years ago

Giovanni Manghi wrote:

Please test again after the patch has been merged in master.

Hi everybody,
when this patch will be available in the master?
I've today update it to df0d717f92 and the patch is not available
greetings

#12 Updated by Giovanni Manghi over 6 years ago

Antonio Viscomi wrote:

Giovanni Manghi wrote:

Please test again after the patch has been merged in master.

Hi everybody,
when this patch will be available in the master?
I've today update it to df0d717f92 and the patch is not available
greetings

after you will see "merged" here https://github.com/qgis/QGIS/pull/5410

#13 Updated by Antonio Viscomi over 6 years ago

Giovanni Manghi wrote:

Antonio Viscomi wrote:

Giovanni Manghi wrote:

Please test again after the patch has been merged in master.

Hi everybody,
when this patch will be available in the master?
I've today update it to df0d717f92 and the patch is not available
greetings

ok thanks

after you will see "merged" here https://github.com/qgis/QGIS/pull/5410

#14 Updated by Sandro Mani over 6 years ago

The PR is now merged to master.

#15 Updated by Harrissou Santanna over 6 years ago

The patch is against master. Will the fix be backported?

#16 Updated by Sandro Mani over 6 years ago

The PR is much more than a fix, it's a major rework of the plugin, and as such no, it won't be backported.

#17 Updated by Antonio Viscomi over 6 years ago

Sandro Mani wrote:

The PR is now merged to master.

I did a quick test on the plugin in the master
and it work perfectly for the file I attached to the beginning of the topic.
many thanks for the excellent work Sandro Mani

#18 Updated by Giovanni Manghi over 6 years ago

  • Status changed from Feedback to Closed
  • Resolution set to fixed/implemented

Also available in: Atom PDF