Opened 8 years ago
Closed 8 years ago
#13966 closed enhancement (othersoftware)
requirement of bounds in maps
Reported by: | Klumbumbus | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core imagery | Version: | |
Keywords: | template_report | Cc: |
Description
It seems JOSM requires min/max bounds even when a shape is present.
The XML file from ELI has only shapes but no bounds.
This results in two problems:
- When using the ELI index no polygons are drawn at the minimap at Help/Preferences/Imagery
- Copying entries from ELI to JOSM is not simply possible since the JOSM wiki doesn't allow entries without bounds (and it wouldn't work then anyway)
To me it seems the bounds information is superfluous if a shape is present. So could JOSM correctly handle and allow imagery entries which have a shape but no bounds?
URL:http://josm.openstreetmap.de/svn/trunk Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b Last:Changed Date: 2016-11-13 03:19:10 +0100 (Sun, 13 Nov 2016) Build-Date:2016-11-13 02:34:00 Revision:11249 Relative:URL: ^/trunk Identification: JOSM/1.5 (11249 de) Windows 7 32-Bit Memory Usage: 346 MB / 870 MB (127 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=false, -Djava.util.Arrays.useLegacyMergeSort=true, -Djnlpx.heapsize=256m,900m, -Djnlpx.splashport=49669, -Djnlpx.jvm=<java.home>\bin\javaw.exe, -Djnlpx.vmargs=LURqYXZhLnV0aWwuQXJyYXlzLnVzZUxlZ2FjeU1lcmdlU29ydD10cnVlAA==]
Attachments (0)
Change History (8)
comment:1 by , 8 years ago
comment:2 by , 8 years ago
Milestone: | → 16.11 |
---|
follow-up: 4 comment:3 by , 8 years ago
Rather than adapting JOSM I'd suggest to fix the XML conversion script, so that bounds are added - a fix to JOSM nevertheless would require adding bounds, as it will take longer time until all old installations are updated.
I'm not totally sure about internal tests in JOSM, but I hope than before checking all the polygons first the bounds are checked, so that more complex calculations are not done when already outside of the rectangular bounds.
follow-up: 5 comment:4 by , 8 years ago
Replying to stoecker:
Rather than adapting JOSM I'd suggest to fix the XML conversion script, so that bounds are added
Done: https://github.com/osmlab/editor-layer-index/commit/846fcc2e285795cd1820d7455e30daaead6730d5
follow-up: 6 comment:5 by , 8 years ago
Replying to simon04:
Done: https://github.com/osmlab/editor-layer-index/commit/846fcc2e285795cd1820d7455e30daaead6730d5
Great. One more thing: Could you change the indentation from 2 spaces to 4 spaces at https://github.com/osmlab/editor-layer-index/blob/gh-pages/imagery.xml. It shouldn't hurt there and would better fit to the JOSM lists.
follow-up: 7 comment:6 by , 8 years ago
Replying to Klumbumbus:
Replying to simon04:
Done: https://github.com/osmlab/editor-layer-index/commit/846fcc2e285795cd1820d7455e30daaead6730d5
Great. One more thing: Could you change the indentation from 2 spaces to 4 spaces at https://github.com/osmlab/editor-layer-index/blob/gh-pages/imagery.xml. It shouldn't hurt there and would better fit to the JOSM lists.
But keep in mind, that this ouput is designed to be input to JOSM. Our maps output is also totally stripped to reduce size. Only the wiki has indentation.
comment:7 by , 8 years ago
Replying to stoecker:
Replying to Klumbumbus:
Replying to simon04:
Done: https://github.com/osmlab/editor-layer-index/commit/846fcc2e285795cd1820d7455e30daaead6730d5
Great. One more thing: Could you change the indentation from 2 spaces to 4 spaces at https://github.com/osmlab/editor-layer-index/blob/gh-pages/imagery.xml. It shouldn't hurt there and would better fit to the JOSM lists.
But keep in mind, that this ouput is designed to be input to JOSM. Our maps output is also totally stripped to reduce size. Only the wiki has indentation.
OK, it is not really necessary to change this. One can quickly change the indentation in notepad++ for the whole file and copy then from there.
comment:8 by , 8 years ago
Milestone: | 16.11 |
---|---|
Resolution: | → othersoftware |
Status: | new → closed |
This is actually fixed.
There are a lot
imagico.de OSM images for mapping
imageries missing in JOSM. They could quickly be copied from ELI once this problem is fixed.