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.

Sign up for GitHub

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

Jump to bottom

QGIS crashes on zoom in with wheel closer than 1:1 and font marker in style #42270

Closed
erpas opened this issue Mar 16, 2021 · 5 comments · Fixed by #43426
Closed

QGIS crashes on zoom in with wheel closer than 1:1 and font marker in style #42270

erpas opened this issue Mar 16, 2021 · 5 comments · Fixed by #43426
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Crash/Data Corruption High Priority

Comments

@erpas
Copy link
Contributor

erpas commented Mar 16, 2021

Describe the bug

For QGIS project containing a layer styled with a font marker, when zooming in closer than 1:1, QGIS CPU usage jumps to 100% and the application crashes after a while.

This is similar to #32489.

How to Reproduce

  1. Open QGIS project attached.
  2. Zoom in much closer than 1:1 then back to a sensible zoom level several times.
  3. QGIS hangs for a while and then crashes

QGIS and OS versions

QGIS version 3.16.4-Hannover QGIS code revision 654e76b
Compiled against Qt 5.11.2 Running against Qt 5.11.2
Compiled against GDAL/OGR 3.1.4 Running against GDAL/OGR 3.1.4
Compiled against GEOS 3.8.1-CAPI-1.13.3 Running against GEOS 3.8.1-CAPI-1.13.3
Compiled against SQLite 3.29.0 Running against SQLite 3.29.0
PostgreSQL Client Version 11.5 SpatiaLite Version 4.3.0
QWT Version 6.1.3 QScintilla2 Version 2.10.8
Compiled against PROJ 6.3.2 Running against PROJ Rel. 6.3.2, May 1st, 2020
OS Version Windows 10 (10.0)
Active python plugins db_manager; MetaSearch; processing
@erpas erpas added the Bug Either a bug report, or a bug fix. Let's hope for the latter! label Mar 16, 2021
@erpas
Copy link
Contributor Author

erpas commented Mar 16, 2021

carto_sym.zip

@gioman
Copy link
Contributor

gioman commented Mar 16, 2021

@erpas confirmed on 3.16.4 but bot on 3.18.0, can you check?

@roya0045
Copy link
Contributor

roya0045 commented Mar 16, 2021 •
edited
Loading

Pretty sure this is a duplicate.

See #34056 #33902

@erpas
Copy link
Contributor Author

erpas commented Mar 16, 2021

@gioman sorry, I don't have 3.18 at hand now. Although I was told by colleague that 3.18 does not crash on this project.

@danzig666
Copy link

@erpas confirmed on 3.16.4 but bot on 3.18.0, can you check?

Just checked on 3.18.0. Crashes too.

@nyalldawson nyalldawson mentioned this issue May 28, 2021
Merged
nyalldawson added a commit to nyalldawson/QGIS that referenced this issue May 28, 2021
@nyalldawson
Don't try to render font marker symbols in massive font sizes
a87e3d2 
Instead, scale the painter after the font exceeds a certain threshold.
The end result looks the same, but should avoid issues like the
crash described in qgis#42270

Fixes qgis#42270
nyalldawson added a commit that referenced this issue May 29, 2021
@nyalldawson
Don't try to render font marker symbols in massive font sizes
12d1078 
Instead, scale the painter after the font exceeds a certain threshold.
The end result looks the same, but should avoid issues like the
crash described in #42270

Fixes #42270
nyalldawson added a commit to nyalldawson/QGIS that referenced this issue May 30, 2021
@nyalldawson
Don't try to render font marker symbols in massive font sizes
edfc5ec 
Instead, scale the painter after the font exceeds a certain threshold.
The end result looks the same, but should avoid issues like the
crash described in qgis#42270

Fixes qgis#42270

(cherry picked from commit a87e3d2)
@nyalldawson nyalldawson mentioned this issue May 30, 2021
Merged
nyalldawson added a commit that referenced this issue May 31, 2021
@nyalldawson
Don't try to render font marker symbols in massive font sizes
cd8d418 
Instead, scale the painter after the font exceeds a certain threshold.
The end result looks the same, but should avoid issues like the
crash described in #42270

Fixes #42270

(cherry picked from commit a87e3d2)
nyalldawson added a commit that referenced this issue May 31, 2021
@nyalldawson
Don't try to render font marker symbols in massive font sizes
10bea7e 
Instead, scale the painter after the font exceeds a certain threshold.
The end result looks the same, but should avoid issues like the
crash described in #42270

Fixes #42270

(cherry picked from commit a87e3d2)
nyalldawson added a commit that referenced this issue Jun 14, 2021
@nyalldawson
Don't try to render font marker symbols in massive font sizes
ba29f22 
Instead, scale the painter after the font exceeds a certain threshold.
The end result looks the same, but should avoid issues like the
crash described in #42270

Fixes #42270

(cherry picked from commit a87e3d2)
nyalldawson added a commit that referenced this issue Jun 21, 2021
@nyalldawson
Don't try to render font marker symbols in massive font sizes
3d04f9e 
Instead, scale the painter after the font exceeds a certain threshold.
The end result looks the same, but should avoid issues like the
crash described in #42270

Fixes #42270

(cherry picked from commit a87e3d2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Crash/Data Corruption High Priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't try to render font marker symbols in massive font sizes nyalldawson/QGIS
4 participants
@gioman @erpas @danzig666 @roya0045