Modify ↓

Opened 5 years ago

Closed 4 years ago

#17266 closed defect (fixed)

[Patch RFC] "Download along" does not increment status popup

Reported by: wambacher Owned by: team
Priority: minor Milestone: 20.06
Component: Core Version: tested
Keywords: download along Cc:

Description (last modified by GerdP)

The popup "Download along" does not increment the counter while downloading.

Reproduce: Select all elements of a boundary which has a lot of members like ​https://www.openstreetmap.org/relation/2029050. Then "download along".

current version: Josm 14620, plugin 34867
snapshot

Attachments (5)

download_along.png​ (31.1 KB ) - added by wambacher 5 years ago.
snapshot
wrongAreas.PNG​ (94.8 KB ) - added by GerdP 5 years ago.
Peek 2019-01-28 13-03.gif​ (1.2 MB ) - added by wambacher 5 years ago.
animated gif
17266.patch​ (449 bytes ) - added by GerdP 5 years ago.
17266-v2.patch​ (472 bytes ) - added by GerdP 5 years ago.

Download all attachments as: .zip

Change History (39)

by wambacher, 5 years ago

Attachment: download_along.png​ added

snapshot

by GerdP, 5 years ago

Attachment: wrongAreas.PNG​ added

comment:1 by GerdP, 5 years ago

Description: modified (diff)

I cannot reproduce that with settings
downloadAlongWay.area 20.0
downloadAlongWay.distance 50.0
downloadAlongWay.download.gps false
What are your settings?

What I see is that the plugin seems to download areas which don't make sense, see .

comment:2 by GerdP, 5 years ago

I'll create a new ticket for my problem. Maybe yours is related, the calculation of the download areas is wrong.

comment:3 by GerdP, 5 years ago

Description: modified (diff)

comment:4 by GerdP, 5 years ago

see #17267. Please retry again with the new version later, the automatic build process is still working.

comment:5 by wambacher, 5 years ago

distance 200m, area=5 km

but that does not matter anyhow.

when i do the download, the result are 140 steps, but the counter permanently says "1/140".

and yes, sometimes the results are strange - but that is not the problem.

walter

hi gerd, ich hab verschiedene parameter für den download verwendet, es macht aber garnix aus. ich bekomme diesen Popup und der ändert sich solange nicht, bis die Aufgabe erledigt ist. damit weiss ich nicht, wie lange das noch dauert. also: bekommst du diesen Popup überhaupt?

Last edited 5 years ago by wambacher (previous) (diff)

comment:6 by GerdP, 5 years ago

I see the popup and the counter increases. Are you sure that you use r14620 without any patches?

comment:7 by wambacher, 5 years ago

yes, i do.

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2018-12-31 15:09:58 +0100 (Mon, 31 Dec 2018)
Build-Date:2018-12-31 14:24:10
Revision:14620
Relative:URL: /trunk

wambacher@server2:~/osm/maven/missing_boundaries$ josm
Using /usr/lib/jvm/java-11-openjdk-amd64/bin/java to execute josm.
2019-01-28 12:44:03.811 INFORMATION: Protokollierungsgrad ist bei INFORMATION (INFO, 800)
2019-01-28 12:44:07.573 INFORMATION: Erweiterung 'jaxb' wird geladen (Version 34678)
2019-01-28 12:44:07.577 INFORMATION: Erweiterung 'log4j' wird geladen (Version 34527)
Warning: Nashorn engine is planned to be removed from a future JDK release
2019-01-28 12:44:10.301 INFORMATION: GET https://josm.openstreetmap.de/wiki/De:StartupPage -> 200
2019-01-28 12:44:10.760 WARNUNG: Not a single layer for the name 'AeroWest': []
2019-01-28 12:44:10.760 WARNUNG: Not a single layer for the name 'Bing Sat': []
2019-01-28 12:44:10.761 WARNUNG: Not a single layer for the name 'Bing Sat': []
2019-01-28 12:44:10.761 WARNUNG: Not a single layer for the name 'Bing Sat': []
2019-01-28 12:44:10.762 WARNUNG: Not a single layer for the name 'Bing Sat': []
2019-01-28 12:44:10.762 WARNUNG: Not a single layer for the name 'Bing Sat': []
2019-01-28 12:44:10.762 WARNUNG: Not a single layer for the name 'Bing Sat': []
2019-01-28 12:44:10.763 WARNUNG: Not a single layer for the name 'Bing Sat': []
2019-01-28 12:44:10.763 WARNUNG: Not a single layer for the name 'Bing Sat': []
2019-01-28 12:44:10.764 WARNUNG: Not a single layer for the name 'wms.wambachers-osm.website: Stadtteilgrenzen_KreisRE': []
2019-01-28 12:44:12.072 SCHWERWIEGEND: Fehler beim Laden des Bildes 'dialogs/up.png'
2019-01-28 12:44:12.073 WARNUNG: Sidewalk both: Could not get presets icon dialogs/up.png
2019-01-28 12:44:12.074 SCHWERWIEGEND: Fehler beim Laden des Bildes 'dialogs/previous.png'
2019-01-28 12:44:12.074 WARNUNG: Sidewalk left: Could not get presets icon dialogs/previous.png
2019-01-28 12:44:12.075 SCHWERWIEGEND: Fehler beim Laden des Bildes 'dialogs/next.png'
2019-01-28 12:44:12.075 WARNUNG: Sidewalk right: Could not get presets icon dialogs/next.png
2019-01-28 12:44:12.076 SCHWERWIEGEND: Fehler beim Laden des Bildes 'presets/way_unclassified.png'
2019-01-28 12:44:12.076 WARNUNG: Sidewalk no: Could not get presets icon presets/way_unclassified.png
2019-01-28 12:44:12.078 SCHWERWIEGEND: Fehler beim Laden des Bildes 'presets/bus_halt.png'
2019-01-28 12:44:12.079 WARNUNG: Bus Haltestelle/Schild: Could not get presets icon presets/bus_halt.png
Warning: Nashorn engine is planned to be removed from a future JDK release
2019-01-28 12:44:12.332 WARNUNG: Cannot lock cache directory. Will not use disk cache
2019-01-28 12:44:12.397 INFORMATION: Could not load tool definition continuosdownload/activate
2019-01-28 12:44:12.550 INFORMATION: Erweiterung 'reverter' wird geladen (Version 34867)
2019-01-28 12:44:12.567 INFORMATION: RemoteControl: adding command "revert_changeset" (handled by RevertChangesetHandler)
2019-01-28 12:44:12.570 INFORMATION: Erweiterung 'apache-commons' wird geladen (Version 34506)
2019-01-28 12:44:12.581 INFORMATION: Erweiterung 'reltoolbox' wird geladen (Version 34867)
2019-01-28 12:44:12.604 INFORMATION: Erweiterung 'jts' wird geladen (Version 34524)
2019-01-28 12:44:12.606 INFORMATION: Erweiterung 'ejml' wird geladen (Version 34389)
2019-01-28 12:44:12.628 INFORMATION: Erweiterung 'geotools' wird geladen (Version 34513)
2019-01-28 12:44:12.703 WARNUNG: No configuration settings found. Using hardcoded default values for all pools.
2019-01-28 12:44:13.165 INFORMATION: Warp/affine reduction enabled: true
2019-01-28 12:44:13.988 INFORMATION: dataFileCache open start
2019-01-28 12:44:14.219 INFORMATION: Erweiterung 'utilsplugin2' wird geladen (Version 34867)
2019-01-28 12:44:14.375 INFORMATION: Erweiterung 'HouseNumberTaggingTool' wird geladen (Version 34867)
2019-01-28 12:44:14.382 INFORMATION: Erweiterung 'opendata' wird geladen (Version 34867)
2019-01-28 12:44:14.411 INFORMATION: Erweiterung 'RoadSigns' wird geladen (Version 34867)
2019-01-28 12:44:14.465 INFORMATION: Erweiterung 'dataimport' wird geladen (Version 34749)
2019-01-28 12:44:14.470 INFORMATION: Erweiterung 'continuosDownload' wird geladen (Version 82)
2019-01-28 12:44:14.475 INFORMATION: Erweiterung 'pdfimport' wird geladen (Version 34867)
2019-01-28 12:44:14.483 INFORMATION: Erweiterung 'namemanager' wird geladen (Version 34867)
2019-01-28 12:44:14.488 INFORMATION: Erweiterung 'terracer' wird geladen (Version 34867)
2019-01-28 12:44:14.502 INFORMATION: Erweiterung 'wikipedia' wird geladen (Version v1.1.1)
2019-01-28 12:44:14.632 INFORMATION: Erweiterung 'PicLayer' wird geladen (Version 34867)
2019-01-28 12:44:14.662 INFORMATION: Erweiterung 'DirectUpload' wird geladen (Version 34867)
2019-01-28 12:44:14.667 INFORMATION: Erweiterung 'geojson' wird geladen (Version 116)
2019-01-28 12:44:14.671 INFORMATION: Erweiterung 'pbf' wird geladen (Version 34576)
2019-01-28 12:44:14.680 INFORMATION: Erweiterung 'photo_geotagging' wird geladen (Version 34867)
2019-01-28 12:44:14.711 INFORMATION: Erweiterung 'OpeningHoursEditor' wird geladen (Version 34867)
2019-01-28 12:44:14.737 INFORMATION: Erweiterung 'poly' wird geladen (Version 34867)
2019-01-28 12:44:14.740 INFORMATION: Erweiterung 'undelete' wird geladen (Version 34867)
2019-01-28 12:44:14.762 INFORMATION: Erweiterung 'buildings_tools' wird geladen (Version 34867)
2019-01-28 12:44:14.788 INFORMATION: Erweiterung 'Tracer2' wird geladen (Version 34867)
2019-01-28 12:44:14.872 INFORMATION: Erweiterung 'download_along' wird geladen (Version 34867)
2019-01-28 12:44:14.874 INFORMATION: Reassigning shortcut 'tools:download_along' from null to 576 because of conflict with Alt+Umschalt+D
2019-01-28 12:44:14.874 INFORMATION: Tastenkürzelkonflikt: 'tools:download_along' wurde von 'continuosdownload:activate' auf 'Alt+Umschalt+F1' verschoben.
2019-01-28 12:44:15.317 WARNUNG: Cannot start IPv4 remotecontrol server on port 8111: Die Adresse wird bereits verwendet (Bind failed)
2019-01-28 12:44:15.317 WARNUNG: Cannot start IPv6 remotecontrol server on port 8111: Die Adresse wird bereits verwendet (Bind failed)
2019-01-28 12:44:15.358 WARNUNG: Cannot start IPv4 remotecontrol https server on port 8112: Die Adresse wird bereits verwendet (Bind failed)
2019-01-28 12:44:15.360 WARNUNG: Cannot start IPv6 remotecontrol https server on port 8112: Die Adresse wird bereits verwendet (Bind failed)
2019-01-28 12:44:15.364 INFORMATION: Message notifier active (checks every 5 minutes)
2019-01-28 12:44:15.381 INFORMATION: Changeset updater active (checks every 60 minutes if open changesets have been closed)
2019-01-28 12:44:15.446 INFORMATION: GET ​http://www.openstreetmap.org/api/0.6/user/details (Anzahl der ungelesenen Nachrichten abrufen) -> 301 (345 B)
2019-01-28 12:44:15.446 INFORMATION: Herunterladen umgelenkt auf '​https://www.openstreetmap.org/api/0.6/user/details'
2019-01-28 12:44:15.724 INFORMATION: GET ​https://www.openstreetmap.org/api/0.6/user/details (Anzahl der ungelesenen Nachrichten abrufen) -> 200 (614 B)
wambacher@server2:~/osm/maven/missing_boundaries$

some minor warnings, but that is not important, i think.

comment:8 by GerdP, 5 years ago

Maybe a problem with the JRE? I use java version "1.8.0_201"

by wambacher, 5 years ago

Attachment: Peek 2019-01-28 13-03.gif​ added

animated gif

comment:9 by wambacher, 5 years ago

added animated gif.

comment:10 by wambacher, 5 years ago

wambacher@server2:~/osm/maven/boundaries_work/alb$ java -version
openjdk version "11.0.1" 2018-10-16
OpenJDK Runtime Environment (build 11.0.1+13-Ubuntu-2ubuntu1)
OpenJDK 64-Bit Server VM (build 11.0.1+13-Ubuntu-2ubuntu1, mixed mode, sharing)
wambacher@server2:~/osm/maven/boundaries_work/alb$

but 2-3 month before running java 1.8 - same problem.

Last edited 5 years ago by wambacher (previous) (diff)

comment:11 by GerdP, 5 years ago

Do you see a progress in other popups? E.g. when you use continousDownload?

comment:12 by GerdP, 5 years ago

I've just tried with 

openjdk version "11.0.1" 2018-10-16
OpenJDK Runtime Environment 18.9 (build 11.0.1+13)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.1+13, mixed mode)

No problems on Windows. Don't know how to make an animated gif, so you have to trust me ;)

comment:13 by simon04, 5 years ago

I cannot reproduce neither.

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2018-12-31 15:09:58 +0100 (Mon, 31 Dec 2018)
Build-Date:2018-12-31 14:24:10
Revision:14620
Relative:URL: ^/trunk

Identification: JOSM/1.5 (14620 en_GB) Linux Arch Linux
Memory Usage: 1338 MB / 3506 MB (849 MB allocated, but free)
Java version: 1.8.0_202-b26, Oracle Corporation, OpenJDK 64-Bit Server VM
VM arguments: [-Djosm.home=<josm.pref>]
Dataset consistency test: No problems found

Plugins:
+ download_along (34869)

@wambacher: Please retry having only the download_along plugin enabled, e.g., using josm.home from Help/CommandLineOptions#Javaoptions. Otherwise this issue seems to be impossible to reproduce/debug.

comment:14 by wambacher, 5 years ago

Bingo: new empty home for josm, the only plugin is "download_along"

---> problem gone. :)

give me some time to find the reason. you can close this ticket and may be, i'll open a new one asap.

Last edited 5 years ago by wambacher (previous) (diff)

comment:15 by GerdP, 5 years ago

Maybe you now have the newer plugin version.

comment:16 by wambacher, 5 years ago

yes, the plugin version was wrong.

old log: 2019-01-28 12:44:14.872 INFORMATION: Erweiterung 'download_along' wird geladen (Version 34867)

new log: 2019-01-28 14:12:20.715 INFORMATION: Erweiterung 'download_along' wird geladen (Version 34869)

But the problem is not gone in the live josm. only test josm is running fine.

comment:17 by wambacher, 5 years ago

activated all plugins in test josm --> no problem

now i'll check preferences and local profile :(

comment:18 by wambacher, 5 years ago

Using my old preferences.xml brings the problem back :(

will check it in the next 1-2 days.

comment:19 by simon04, 5 years ago

Resolution: → worksforme
Status: new → closed

Please reopen this ticket if you find a way to reproduce it w/ a fresh config.

comment:20 by wambacher, 5 years ago

It's crasy.

OSM Server address = default (​https://api.openstreetmap.org) --> OK
OSM server address = ​http://api.openstreetmap.org) --> problem

and yes, my josm was using http :(

using http josm does a reconnect to https. may be that is the reason?

2019-01-30 10:42:43.826 INFORMATION: GET ​http://www.openstreetmap.org/api/0.6/map?bbox=28.0472217,51.5538382,28.0741319,51.5556308 -> 301 (385 B)
2019-01-30 10:42:43.827 INFORMATION: Herunterladen umgelenkt auf '​https://www.openstreetmap.org/api/0.6/map?bbox=28.0472217,51.5538382,28.0741319,51.5556308'
2019-01-30 10:42:43.983 INFORMATION: GET ​https://www.openstreetmap.org/api/0.6/map?bbox=28.0472217,51.5538382,28.0741319,51.5556308 -> 200
2019-01-30 10:42:44.094 INFORMATION: GET ​http://www.openstreetmap.org/api/0.6/map?bbox=28.03487,51.5715121,28.0741319,51.5733135 -> 301 (383 B)
2019-01-30 10:42:44.095 INFORMATION: Herunterladen umgelenkt auf '​https://www.openstreetmap.org/api/0.6/map?bbox=28.03487,51.5715121,28.0741319,51.5733135'
2019-01-30 10:42:44.149 INFORMATION: GET ​https://www.openstreetmap.org/api/0.6/map?bbox=28.03487,51.5715121,28.0741319,51.5733135 -> 200

Version 1, edited 5 years ago by wambacher (previous) (next) (diff)

comment:21 by wambacher, 5 years ago

Resolution: worksforme
Status: closed → reopened

comment:22 by GerdP, 5 years ago

For me, ​http://api.openstreetmap.org doesn't work at all. One proably has to change more than that.

comment:24 by GerdP, 5 years ago

I get an error when I use that address:

2019-01-30 18:04:18.161 FINE: Bad OAuth request.
2019-01-30 18:04:18.188 SEVERE: org.openstreetmap.josm.io.OsmApiException: ResponseCode=400, Error Header=<Bad OAuth request.>
org.openstreetmap.josm.io.OsmApiException: ResponseCode=400, Error Header=<Bad OAuth request.>

comment:25 by GerdP, 5 years ago

I was finally able to reproduce it with http://master.apis.dev.openstreetmap.org/api. Working on it...

by GerdP, 5 years ago

Attachment: 17266.patch​ added

comment:26 by GerdP, 5 years ago

With 17266.patch it works for both http and https.
Please review:
Class HttpClient starts a new subtask monitor in line 133. I found no place where this is finished. The patch changes this.
No idea if there is a case where this causes trouble. So far I found none.

comment:27 by GerdP, 5 years ago

Arg, patch doesn't work with undelete plugin when undeleting multiple objects using https.
So, sorry, that's too complex for me :(

by GerdP, 5 years ago

Attachment: 17266-v2.patch​ added

comment:28 by GerdP, 5 years ago

With v2 it seems to work better. Now progressMonitor.finishTask() is only called if the connection was not established.
That's probably what is missing because the redirect calls another connect() which means one more progressMonitor.beginTask(tr("Contacting Server..."), 1).

Hope this helps, I'll not commit this since my knowledge about this stuff is poor. With the first patch you get much uglier problems than a "missing progress" ;)

comment:29 by GerdP, 5 years ago

Please can somebody look at patch v2?

comment:30 by Klumbumbus, 5 years ago

Component: Plugin download_along → Core

comment:31 by Don-vip, 5 years ago

Keywords: download along added
Summary: Plugin "Download along" does not increment status popup → "Download along" does not increment status popup

comment:32 by GerdP, 4 years ago

Summary: "Download along" does not increment status popup → [Patch RFC] "Download along" does not increment status popup

I'd like to commit the patch. Anybody willing to review this one-liner?

comment:33 by simon04, 4 years ago

Milestone: → 20.06

The patch looks good. In normal operation, finishTask is called in org.openstreetmap.josm.io.StreamProgressUpdater#finishTask. Go ahead :-)

comment:34 by GerdP, 4 years ago

Resolution: → fixed
Status: reopened → closed

In 16551/josm:

fix #17266: "Download along" does not increment status popup

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.