grass icon indicating copy to clipboard operation
grass copied to clipboard

grass.jupyter: fix TimeSeriesMap layer rendering

Open petrasovaa opened this issue 7 months ago • 2 comments

Addresses #5304. It required larger changes, moving more code to the BaseSeriesMap class.

It also fixes rendering vector time series and newly enables to pass d.rast/d.vect variables to TimeSeriesMap.add_raster_series/add_vector_series for symbology.

I tested it with the doc/examples/notebooks.

petrasovaa avatar May 09 '25 19:05 petrasovaa

If you want me to review it, I can try tomorrow on a full computer, it's hard to follow the context only on mobile. If anyone else wanted to take a look instead it's fine too.

echoix avatar May 20 '25 01:05 echoix

If you want me to review it, I can try tomorrow on a full computer, it's hard to follow the context only on mobile. If anyone else wanted to take a look instead it's fine too.

To properly test it, you need to run some of the example notebooks, but maybe you don't need to do that, I tried to test it.

petrasovaa avatar May 20 '25 02:05 petrasovaa