Modify

Opened 7 months ago

Last modified 7 months ago

#23963 assigned defect

panoviewer

Reported by: wegavision Owned by: GhostFoxSledgehammer
Priority: normal Milestone:
Component: Plugin panoviewer Version:
Keywords: template_report Cc:

Description

What steps will reproduce the problem?

First of all, I would like to praise the fact that the 3D view now works without crashing and zooming also works.
However, it is annoying that the 3D image always jumps back to one direction when you touch it with the right mouse pointer to move it. In my case, this direction is always 270° from the direction straight ahead. You then have to keep reorienting yourself to the point you want to edit.
It always seems to be the case when you go forward to the next image, then the view is maintained, but as soon as you touch the image with the mouse, it jumps to 270°.

What is the expected result?

What happens instead?

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

Revision:19230
Build-Date:2024-10-02 11:41:59

Identification: JOSM/1.5 (19230 de) Windows 10 64-Bit
OS Build number: Windows 10 Home 22H2 (19045)
Memory Usage: 976 MB / 6100 MB (230 MB allocated, but free)
Java version: 21.0.4+7-LTS, Azul Systems, Inc., OpenJDK 64-Bit Server VM
Look and Feel: com.sun.java.swing.plaf.windows.WindowsLookAndFeel
Screen: \Display0 1280x720x32bpp@60Hz (scaling 1.00×1.00)
Maximum Screen Size: 1280×720
Best cursor sizes: 16×16→32×32, 32×32→32×32
System property file.encoding: UTF-8
System property sun.jnu.encoding: Cp1252
Locale info: de_DE
Numbers with default locale: 1234567890 -> 1234567890
VM arguments: [-Djpackage.app-version=1.5.19230, -XX:MaxRAMPercentage=75.0, --add-modules=java.scripting,java.sql,javafx.controls,javafx.media,javafx.swing,javafx.web, --add-exports=java.base/sun.security.action=ALL-UNNAMED, --add-exports=java.desktop/com.sun.imageio.plugins.jpeg=ALL-UNNAMED, --add-exports=java.desktop/com.sun.imageio.spi=ALL-UNNAMED, --add-opens=java.base/java.lang=ALL-UNNAMED, --add-opens=java.base/java.nio=ALL-UNNAMED, --add-opens=java.base/jdk.internal.loader=ALL-UNNAMED, --add-opens=java.base/jdk.internal.ref=ALL-UNNAMED, --add-opens=java.desktop/javax.imageio.spi=ALL-UNNAMED, --add-opens=java.desktop/javax.swing.text.html=ALL-UNNAMED, --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED, -Djpackage.app-path=%UserProfile%\AppData\Local\JOSM\JOSM.exe]
Dataset consistency test: No problems found

Plugins:
+ HouseNumberTaggingTool (36226)
+ Mapillary (1778)
+ RoadSigns (36343)
+ apache-commons (36282)
+ buildings_tools (36344)
+ easypresets (1623509627)
+ ejml (36176)
+ jaxb (36118)
+ jogl (1.2.3)
+ log4j (36273)
+ lwjgl (36176)
+ lwjgl-windows (36176)
+ measurement (36344)
+ panoviewer (0.9.0)
+ photo_geotagging (36343)
+ tageditor (36344)
+ terracer (36344)
+ turnlanes-tagging (1727811971)
+ turnrestrictions (36344)
+ utilsplugin2 (36344)

Tagging presets:
+ <josm.pref>/EasyPresets.xml

Map paint styles:
- https://josm.openstreetmap.de/josmfile?page=Styles/BuildingColors&zip=1
- https://raw.githubusercontent.com/species/josm-preset-traffic_sign_direction/master/direction.mapcss
- https://josm.openstreetmap.de/josmfile?page=Styles/Enhanced_Lane_and_Road_Attributes&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/FixmeAndNote&zip=1
- https://raw.githubusercontent.com/yopaseopor/traffic_signs_style_JOSM/master/DE.zip
- https://josm.openstreetmap.de/josmfile?page=Styles/AddressValidator&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Styles/Direction&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/Building_Levels_Labels&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Styles/SimpleBuildingTags&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/Potlatch2&zip=1

Last errors/warnings:
- 00365.301 W: java.io.IOException: Attribution is not loaded yet
- 00365.301 W: java.io.IOException: Attribution is not loaded yet
- 00365.301 W: java.io.IOException: Attribution is not loaded yet
- 00365.302 W: java.io.IOException: Attribution is not loaded yet
- 00365.302 W: java.io.IOException: Attribution is not loaded yet
- 00365.302 W: java.io.IOException: Attribution is not loaded yet
- 00365.303 W: java.io.IOException: Attribution is not loaded yet
- 00365.303 W: java.io.IOException: Attribution is not loaded yet
- 00365.304 W: java.io.IOException: Attribution is not loaded yet
- 00365.305 W: java.io.IOException: Attribution is not loaded yet

Attachments (0)

Change History (5)

comment:1 by stoecker, 7 months ago

Component: CorePlugin panoviewer
Owner: changed from team to GhostFoxSledgehammer

I assume the correct place for reports is here: https://github.com/GhostFoxSledgehammer/panoviewer-josm

I assigned the ticket to the author, you'll see if he responds here or not.

comment:2 by GhostFoxSledgehammer, 7 months ago

Status: newassigned

comment:3 by GhostFoxSledgehammer, 7 months ago

Hi,
I've updated the plugin, it should change direction immediately when you go forward to the next image instead of waiting for mouse. This is still inconsistent with the default viewer.
What should be the expected behaviour here, should it be like the default viewer, where the direction is maintained even when image is changed or this is fine?

comment:4 by wegavision, 7 months ago

Hello.
Unfortunately, it's even worse this way, because you usually want to look straight ahead or behind when it comes to the road and traffic. Now you always jump (at least in the pictures I took with my Gopro Max) to the 270° position (seen from the direction of travel) and you have to correct the picture every time.
It would be ideal if you could use the exact angle from the last image when switching to the next image. That's how it used to be, except that it changed when you wanted to move the image again with the mouse.

comment:5 by GhostFoxSledgehammer, 7 months ago

Hi,
I've removed direction resetting after switching to next image, it should work as expected now;

Modify Ticket

Change Properties
Set your email in Preferences
Action
as assigned The owner will remain GhostFoxSledgehammer.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from GhostFoxSledgehammer to the specified user. Next status will be 'new'.
Next status will be 'needinfo'. The owner will be changed from GhostFoxSledgehammer to wegavision.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.

Add Comment


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