qgis / QGIS

QGIS is a free, open source, cross platform (lin/win/mac) geographical information system (GIS)
https://qgis.org
GNU General Public License v2.0
10.09k stars 2.93k forks source link

Abnormal RAM usage by geometry check #27832

Closed qgib closed 5 years ago

qgib commented 5 years ago

Author Name: Antonio Viscomi (@Antoviscomi) Original Redmine Issue: 20010 Affected QGIS version: 3.2.3 Redmine category:c++_plugins/geometry_checker


Hi all, I must perform geometry check in a polygon layer... I setup the checker than I observe a Ram usage of about 30GB

I attach two screenshots

Best regards


qgib commented 5 years ago

Author Name: Giovanni Manghi (@gioman)


Why is tagged as a regression? You does not observe this leak in previous releases like 2.18?


qgib commented 5 years ago

Author Name: Antonio Viscomi (@Antoviscomi)


Giovanni Manghi wrote:

Why is tagged as a regression? You does not observe this leak in previous releases like 2.18?

in 2.18 it not works properly with the same file in previous version of QGIS 3 (but with only 16GB of ram) it worked fine (ram usage about 6-7GB)

qgib commented 5 years ago

Author Name: Giovanni Manghi (@gioman)


in previous version of QGIS 3 (but with only 16GB of ram) it worked fine (ram usage about 6-7GB)

same input file and parameters?

qgib commented 5 years ago

Author Name: Antonio Viscomi (@Antoviscomi)


Input file is huge but I can provide it Tomorrow, For settings in geometry checker how should I can share them? With a screenshot?

qgib commented 5 years ago

Author Name: Antonio Viscomi (@Antoviscomi)


Here are the data i tried to check

https://drive.google.com/file/d/16_PNEWJvXyU0gLig1AncZptZ4OVlCXM-/view?usp=sharing

qgib commented 5 years ago

Author Name: Antonio Viscomi (@Antoviscomi)


Let me know how i can share settings

qgib commented 5 years ago

Author Name: Giovanni Manghi (@gioman)


Antonio Viscomi wrote:

Input file is huge but I can provide it Tomorrow, For settings in geometry checker how should I can share them? With a screenshot?

that was not my question: I asked if the difference in RAM usage amount was noticed using the very same input and configurations in the two QGIS versions you describe.

The dataset is anyway useful, thanks.

qgib commented 5 years ago

Author Name: Antonio Viscomi (@Antoviscomi)


Ok the settings in Italian language are in screenshots attached

Thanks a lot

qgib commented 5 years ago

Author Name: Antonio Viscomi (@Antoviscomi)


Giovanni Manghi wrote:

Antonio Viscomi wrote:

Input file is huge but I can provide it Tomorrow, For settings in geometry checker how should I can share them? With a screenshot?

that was not my question: I asked if the difference in RAM usage amount was noticed using the very same input and configurations in the two QGIS versions you describe.

The dataset is anyway useful, thanks.

Yes dataset is the same in previous and current version (qgis3) for qgis2 when i launch the check qgis it crashes and it shut down

qgib commented 5 years ago

Author Name: Antonio Viscomi (@Antoviscomi)


Antonio Viscomi wrote:

Giovanni Manghi wrote:

Antonio Viscomi wrote:

Input file is huge but I can provide it Tomorrow, For settings in geometry checker how should I can share them? With a screenshot?

that was not my question: I asked if the difference in RAM usage amount was noticed using the very same input and configurations in the two QGIS versions you describe.

The dataset is anyway useful, thanks.

Yes dataset is the same in previous and current version (qgis3) for qgis2 when i launch the check qgis it crashes and it shut down

Parameters are also the same

qgib commented 5 years ago

Author Name: Antonio Viscomi (@Antoviscomi)


Today I've retry to perform geometry check with the same parameter and the same data but with a clean user profile

well: the RAM usage was back to normal at about 6.7GB then I suppose that my old corrupt profile caused abnormal usage

than this ticket could be closed

thanks a lot for your attention

qgib commented 5 years ago

Author Name: Giovanni Manghi (@gioman)