Modify

Opened 11 years ago

Closed 6 years ago

#8256 closed defect (fixed)

GPS data download not working in background with Mapdust active

Reported by: malenki Owned by: jBeata
Priority: normal Milestone:
Component: Plugin mapdust Version: latest
Keywords: background Cc:

Description

How to reproduce:
Install the mapdust plugin.
Go to an area with a lot of gps data (you may try this: region).
Click the button to download the GPS data in background.
Zoom in or out.
Result: the gps data download dialog shows up again

Attachments (0)

Change History (7)

comment:1 by Don-vip, 11 years ago

Keywords: background added

in reply to:  description comment:2 by jBeata, 11 years ago

Replying to malenki:

How to reproduce:
Install the mapdust plugin.
Go to an area with a lot of gps data (you may try this: region).
Click the button to download the GPS data in background.
Zoom in or out.
Result: the gps data download dialog shows up again

Last edited 11 years ago by jBeata (previous) (diff)

comment:3 by jBeata, 11 years ago

I check this issue, and it seams that the GPS "background" download process blocks all the other "download" processes. When you zoom/in out and the Mapdust plugin is activated, the data from your current view is downloading.

comment:4 by malenki, 10 years ago

Summary: GPS data download not working in backgroundGPS data download not working in background with Mapdust active

comment:5 by malenki, 8 years ago

Just want to mention that the issue still is there.

comment:6 by Don-vip, 7 years ago

MapDust code:

        Main.worker.execute(new Runnable() {
            @Override
            public void run() {
                updateMapdustData();
            }
        });

JOSM ProgressMonitorExecutor code:

    @Override
    public void execute(Runnable command) {
        if (Main.currentProgressMonitor != null) {
            //TODO show only if this can't be in background or better if always in background is not checked
            Main.currentProgressMonitor.showForegroundDialog();
        }
        super.execute(command);
    }

MapDust should probably use its own ThreadPoolExecutor, like Mapillary plugin for example.

comment:7 by Don-vip, 6 years ago

Resolution: fixed
Status: newclosed

Fixed in [o34214:34215].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain jBeata.
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.