Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#21170 closed defect (fixed)

Mapillary plugin: multiple images "selected"

Reported by: richlv Owned by: taylor.smock
Priority: normal Milestone:
Component: Plugin mapillary Version: tested
Keywords: template_report Cc:

Description

What steps will reproduce the problem?

  1. Download Mapillary sequence with images, close to one another.
  2. Zoom to the lowest level where individual images are still shown as circles.
  3. Move the mouse cursor over some image cluster. Notice multiple images highlighted.
  4. Click - notice multiple images being "selected".

What is the expected result?

One image selected.

What happens instead?

Multiple get "selected" (only one shown in the image view).

Please provide any additional information below. Attach a screenshot if possible.

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2021-07-12 02:41:41 +0200 (Mon, 12 Jul 2021)
Build-Date:2021-07-12 00:42:49
Revision:18004
Relative:URL: ^/trunk

Identification: JOSM/1.5 (18004 en_GB) Mac OS X 10.15.7
OS Build number: Mac OS X 10.15.7 (19H1217)
Memory Usage: 2753 MB / 3641 MB (1487 MB allocated, but free)
Java version: 1.8.0_301-b09, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Look and Feel: com.apple.laf.AquaLookAndFeel
Screen: Display 69733382 1680×1050 (scaling 1.00×1.00)
Maximum Screen Size: 1680×1050
Best cursor sizes: 16×16→16×16, 32×32→32×32
System property file.encoding: UTF-8
System property sun.jnu.encoding: UTF-8
Locale info: en_GB
Numbers with default locale: 1234567890 -> 1234567890
VM arguments: [-Djnlp.application.href=https://josm.openstreetmap.de/download/josm.jnlp, -Djava.util.Arrays.useLegacyMergeSort=true, -Djnlp.tk=awt, -Djnlpx.jvm=<java.home>/bin/java, -Djnlpx.splashport=-1, -Djnlpx.home=<java.home>/bin, -Djnlpx.remove=false, -Djnlpx.offline=false, -Djnlpx.relaunch=true, -Djnlpx.session.data=/var/folders/nl/flqxqsmj5q963r7tcnfrdt3c0000gn/T/session7264544933387885260, -Djnlpx.heapsize=NULL,NULL, -Djava.security.policy=file:<java.home>/lib/security/javaws.policy, -DtrustProxy=true, -Djnlpx.origFilenameArg=/Users/richlv/Library/Application Support/Oracle/Java/Deployment/cache/6.0/56/1ee8cfb8-72e8e992, -Dsun.awt.warmup=true, -Djava.security.manager]
Dataset consistency test: No problems found

Plugins:
+ HouseNumberTaggingTool (35640)
+ InfoMode (35543)
+ Mapillary (2.0.0-alpha.28-dirty)
+ PicLayer (1.0.1)
+ apache-commons (35524)
+ apache-http (35589)
+ dataimport (35640)
+ ejml (35458)
+ geotools (35458)
+ imagery_offset_db (35640)
+ javafx-osx (35655)
+ jaxb (35543)
+ jna (35662)
+ jts (35458)
+ measurement (35640)
+ opendata (35640)
+ pbf (35720)
+ photo_geotagging (35783)
+ reverter (35732)
+ utilsplugin2 (35792)

Map paint styles:
+ /Users/richlv/Desktop/ChangeFontSize.mapcss
- https://josm.openstreetmap.de/josmfile?page=Styles/Coloured_Streets&zip=1

Last errors/warnings:
- 00018.069 W: Failed to load Mappaint styles from '/Users/richlv/Desktop/ChangeFontSize.mapcss'. Exception was: java.nio.file.NoSuchFileException: /Users/richlv/Desktop/ChangeFontSize.mapcss
- 00018.070 E: java.nio.file.NoSuchFileException: /Users/richlv/Desktop/ChangeFontSize.mapcss
- 00018.078 E: java.nio.file.NoSuchFileException: /Users/richlv/Desktop
- 00018.078 W: Initializing map style /Users/richlv/Desktop/ChangeFontSize.mapcss completed in 10 ms (1 errors, 0 warnings)
- 00350.544 W: Conflicts detected - <html>There was 1 conflict detected.</html>

Attachments (0)

Change History (4)

comment:1 by taylor.smock, 4 years ago

Thanks for poking me -- I had been intending to tweak the distance for that functionality a bit, but I forgot. :(

I'll try to get to this as soon as I finish my current train of thought for another project.

comment:2 by taylor.smock, 4 years ago

Resolution: fixed
Status: newclosed

Fixed in 16a8557df9d6476952d0269f901c86bebb53ccce. I haven't done a release yet (I've got to wait for the pipeline to finish).

I should have an update out in the next hour or two. If I don't, poke me (vorpalblade-kaart on IRC). :)

comment:3 by taylor.smock, 4 years ago

Fix released in v2.0.0-alpha.29. The updated version should show up in < 10 minutes (see PluginsSource).

comment:4 by richlv, 4 years ago

Oh yay, thank you for the quick fix - will try it out soon :)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain taylor.smock.
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 .
 
Note: See TracTickets for help on using tickets.