Opened 7 years ago

Closed 3 years ago

#2234 closed enhancement (fixed)

wxGUI d.mon and g.gui monitor rendering is slow

Reported by: wenzeslaus Owned by: grass-dev@…
Priority: minor Milestone: 7.4.1
Component: wxGUI Version: unspecified
Keywords: rendering, display Cc:
CPU: Unspecified Platform: All

Description

Map rendering in wxGUI (both d.mon wx* and g.gui) is slow, or to be precise, slower than X monitors in GRASS 6. As reported by users, the problem is more visible on larger screens (with large windows) and through ssh -X.

Rendering works so setting type to enhancement and setting priority to minor since wxGUI is usable (it just could be faster), you can compile GRASS 7 with X monitor support (if I'm correct) and you can use other solutions such as PNG driver. Anyway, it would be awesome if rendering would be faster.

Originally reported in #1719.

Feel free to link related tickets and mailing list discussions here.

Change History (15)

comment:1 Changed 5 years ago by martinl

Milestone: 7.0.07.0.5

comment:2 Changed 5 years ago by martinl

Should be better in GRASS 7.2. Could you please test?

comment:4 Changed 5 years ago by sbl

Yes, thanks. There is a significant difference between 7.0.4svn and fresh 7.2 esp. at larger scale (when I zoom in).

When rendering full map extent the speed-up is less noticeable, though there is a speed-up there too. For my map:

1m 29s in GRASS 7.0.4 and

1m 12s in GRASS 7.2

And a big plus is that the rest of the UI remains responsive during rendering...

comment:5 in reply to:  4 ; Changed 5 years ago by martinl

Replying to sbl:

There is a significant difference between 7.0.4svn and fresh 7.2 esp. at larger scale (when I zoom in).

well, the main difference between 7.0 and 7.2 is that rendering system (speaking about wxGUI + d.mon wx0) in the newer version is multi-threaded. So you should feel difference when your map composition consists more map layers and your computer has more cores ;-) There are no major changes in display drivers itself.

comment:6 Changed 5 years ago by martinl

Milestone: 7.0.57.3.0

comment:7 Changed 5 years ago by martinl

Milestone: 7.3.07.4.0

Milestone renamed

comment:8 Changed 5 years ago by neteler

Anything left here? It is wontfix for 7.0.x and done for 7.2.0, right?

(And #2033 is stalled for 3 years...)

comment:9 in reply to:  5 ; Changed 5 years ago by mlennert

Replying to martinl:

Replying to sbl:

There is a significant difference between 7.0.4svn and fresh 7.2 esp. at larger scale (when I zoom in).

well, the main difference between 7.0 and 7.2 is that rendering system (speaking about wxGUI + d.mon wx0) in the newer version is multi-threaded.

Could someone point me to where in the code this was done (and when) ?

comment:10 in reply to:  9 ; Changed 5 years ago by neteler

Replying to mlennert:

Could someone point me to where in the code this was done (and when) ?

Fro a start, check these changes (the work got co-funded from my FEM budget):

comment:11 in reply to:  10 Changed 5 years ago by mlennert

Replying to neteler:

Replying to mlennert:

Could someone point me to where in the code this was done (and when) ?

Fro a start, check these changes (the work got co-funded from my FEM budget):

Ok thanks. This is all in 7.2 already, or ? So this is a big change to announce in the 7.2 news !

comment:12 Changed 5 years ago by mlennert

Just to add to the benchmarking:

Displaying a lidar dataset with +/- 10e6 points in a d.mon wx0, both 7.0 and 7.2 take about 1min16s to run d.vect, but the big difference is afterwards, with 7.2 displaying the data almost immediately afterwards, while 7.0 takes an additional minute to display on the screen.

Great job !

comment:13 Changed 3 years ago by neteler

Milestone: 7.4.07.4.1

Ticket retargeted after milestone closed

comment:14 Changed 3 years ago by martinl

Almost 2 years no activity. Closing, feel free to reopen if needed.

comment:15 Changed 3 years ago by martinl

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.