Modify

Opened 6 years ago

Closed 12 months 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 Changed 6 years ago by Don-vip

Keywords: background added

comment:2 in reply to:  description Changed 6 years ago by jBeata

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 6 years ago by jBeata (previous) (diff)

comment:3 Changed 6 years ago by jBeata

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 Changed 5 years ago by malenki

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

comment:5 Changed 4 years ago by malenki

Just want to mention that the issue still is there.

comment:6 Changed 22 months ago by Don-vip

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 Changed 12 months ago by Don-vip

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.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.