Opened 8 years ago
Closed 8 years ago
#13636 closed defect (fixed)
panning is very slow at high zoom <1m
Reported by: | Owned by: | michael2402 | |
---|---|---|---|
Priority: | critical | Milestone: | 16.10 |
Component: | Core mappaint | Version: | |
Keywords: | template_report gsoc-core regression performance | Cc: | michael2402 |
Description
What steps will reproduce the problem?
- Ich kann die Karte nur sehr sehr langsam verschieben, wenn ich unter 1m zoome.
URL:http://josm.openstreetmap.de/svn/trunk Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b Last:Changed Date: 2016-09-06 00:16:07 +0200 (Tue, 06 Sep 2016) Build-Date:2016-09-05 22:21:00 Revision:10966 Relative:URL: ^/trunk Identification: JOSM/1.5 (10966 de) Windows 7 64-Bit Memory Usage: 542 MB / 910 MB (52 MB allocated, but free) Java version: 1.8.0_101-b13, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM Screen: \Display0 1366x768 Maximum Screen Size: 1366x768 Dataset consistency test: No problems found Plugins: + FixAddresses (32796) + buildings_tools (32944) + jts (32699) + scripting (30730) + terracer (32699) + utilsplugin2 (32815) Last errors/warnings: - W: java.io.IOException: Attribution is not loaded yet - W: java.io.IOException: Attribution is not loaded yet - W: java.io.IOException: Attribution is not loaded yet - W: java.io.IOException: Attribution is not loaded yet - W: java.io.IOException: Attribution is not loaded yet - W: java.io.IOException: Attribution is not loaded yet - W: java.io.IOException: Attribution is not loaded yet - W: java.io.IOException: Attribution is not loaded yet - W: java.io.IOException: Attribution is not loaded yet - W: java.io.IOException: Attribution is not loaded yet
Attachments (2)
Change History (24)
comment:1 by , 8 years ago
Keywords: | regression added |
---|---|
Priority: | normal → critical |
Summary: | Kartenverschiebung bei Zoom unter 1m → panning is very slow at high zoom |
comment:2 by , 8 years ago
Summary: | panning is very slow at high zoom → panning is very slow at high zoom <1m |
---|
comment:3 by , 8 years ago
Cc: | added |
---|---|
Component: | Core → Core mappaint |
Keywords: | gsoc-core added |
Milestone: | → 16.09 |
comment:5 by , 8 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:6 by , 8 years ago
Keywords: | performance added |
---|
follow-up: 15 comment:9 by , 8 years ago
by , 8 years ago
Attachment: | moving_dashes.gif added |
---|
comment:10 by , 8 years ago
Yes, this is one reason I kept this open. It is a bit harder to fix - I cannot set the dash offset on the stroke since the way may have multiple start points. I have an idea on how to fix it (do clamping in a separate step before the way is drawn and when we know the stroke).
I still have performance problems on borders of multipolygon boundaries in some cases. I guessed it was a problem with the contour line for the non-filled areas. But clamping that did not improve the performance. It seems to be related to view caching, it improves if I disable the buffered image in between.
follow-up: 13 comment:12 by , 8 years ago
I'm running 10966 on Windows 10, Java 8u101 x64, and the slow performance in general on high zoom is definitely still present.
comment:13 by , 8 years ago
Replying to Roadsguy:
I'm running 10966 on Windows 10, Java 8u101 x64, and the slow performance in general on high zoom is definitely still present.
Yes, sure. It has been fixed in r11026/r11027.
@michael2402: Besides the moving dashes, is there something to be done, i.e., does this block the next stable release?
comment:15 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Alright. Let's track the moving dashes in #13793.
comment:16 by , 8 years ago
Replying to michael2402:
No, it is just a cosmetic problem.
The moving dashes can be very annoying. It would be nice if it could be fixed in 16.10.
by , 8 years ago
Attachment: | performace.gif added |
---|
comment:17 by , 8 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Sorry to reopen this again, but it seems I didn't completely check this. The performance was improved only a bit in r11026/r11027. Map moving is still extremely slow at high zoom levels (framerate < 1/s). See the screencast, which was created with current latest r11146 (same behavior also with r11030).
With r10823 (i.e. before r10827) I get a fluet panning even on the highest zoom level (however there are moving dashes).
I did't check performance on larger data sets.
URL:http://josm.openstreetmap.de/svn/trunk Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b Last:Changed Date: 2016-10-18 20:32:20 +0200 (Tue, 18 Oct 2016) Build-Date:2016-10-19 01:34:39 Revision:11146 Relative:URL: ^/trunk Identification: JOSM/1.5 (11146 de) Windows 7 32-Bit Memory Usage: 452 MB / 870 MB (85 MB allocated, but free) Java version: 1.8.0_111-b14, Oracle Corporation, Java HotSpot(TM) Client VM Screen: \Display0 1680x1050 Maximum Screen Size: 1680x1050 VM arguments: [-Djava.security.manager, -Djava.security.policy=file:<java.home>\lib\security\javaws.policy, -DtrustProxy=true, -Djnlpx.home=<java.home>\bin, -Djnlpx.origFilenameArg=C:\Program Files\josm-latest-bla.jnlp, -Djnlpx.remove=true, -Djava.util.Arrays.useLegacyMergeSort=true, -Djnlpx.heapsize=256m,900m, -Djnlpx.splashport=56854, -Djnlpx.jvm=<java.home>\bin\javaw.exe, -Djnlpx.vmargs=LURqYXZhLnV0aWwuQXJyYXlzLnVzZUxlZ2FjeU1lcmdlU29ydD10cnVlAA==] Dataset consistency test: No problems found
comment:18 by , 8 years ago
Thanks for reporting.
The one way arrows cause that problem. I fixed it in [11147] by using the same solution as with the dashed lines.
Feel free to re-open if you find any more issues or any features that cause the same problem.
comment:19 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
comment:21 by , 8 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
The repeat-image
MapCSS property is similar. (in StyledMapRenderer.drawRepeatImage()
; e.g. default style for a way tagged natural=cliff
)
I noticed this too. there is a high cpu load just from zooming in and josm hangs (with just a few buildings and a street loaded).