[GRASS-git] [OSGeo/grass] 392568: grass.jupyter: Rename render classes to Map, Map3D...

Caitlin Haedrich noreply at github.com
Thu Apr 14 13:34:03 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/OSGeo/grass
  Commit: 3925684657a3085c92539bb589450c2967f4b48f
      https://github.com/OSGeo/grass/commit/3925684657a3085c92539bb589450c2967f4b48f
  Author: Caitlin Haedrich <69856275+chaedri at users.noreply.github.com>
  Date:   2022-04-14 (Thu, 14 Apr 2022)

  Changed paths:
    M doc/notebooks/basic_example_grass_jupyter.ipynb
    M doc/notebooks/grass_jupyter.ipynb
    M doc/notebooks/hydrology.ipynb
    M doc/notebooks/solar_potential.ipynb
    M doc/notebooks/temporal.ipynb
    M python/grass/jupyter/Makefile
    M python/grass/jupyter/__init__.py
    R python/grass/jupyter/display.py
    R python/grass/jupyter/interact_display.py
    A python/grass/jupyter/interactivemap.py
    A python/grass/jupyter/map.py
    A python/grass/jupyter/map3d.py
    R python/grass/jupyter/render3d.py
    M python/grass/jupyter/reprojection_renderer.py
    R python/grass/jupyter/tests/timeseries_test.py
    A python/grass/jupyter/tests/timeseriesmap_test.py
    R python/grass/jupyter/testsuite/grassrenderer_test.py
    A python/grass/jupyter/testsuite/map3d_test.py
    A python/grass/jupyter/testsuite/map_test.py
    R python/grass/jupyter/testsuite/test_render3d.py
    R python/grass/jupyter/timeseries.py
    A python/grass/jupyter/timeseriesmap.py

  Log Message:
  -----------
  grass.jupyter: Rename render classes to Map, Map3D, TimeSeriesMap (#2305)

Renames spatial visualization (display) classes in grass.jupyter to have common root Map:

- `GrassRenderer` -> `Map` (simple, short, default)
- `Grass3dRenderer` -> `Map3D` (3D as special case, so suffix)
- `TimeSeries` -> `TimeSeriesMap` (adjective for map)

Although map in Python is already an existing function, map is a good match because spatial visualization is the main purpose of these classes, even when you can do non-spatial things with them. Map seems to be the natural choice here, e.g., because some places in GRASS GIS (e.g., map.png as a default filename) and other systems calls these objects maps, too (e.g., folium).

Updated notebooks and changed grass.jupyter filenames to match classes contained.

See also #2283.




More information about the grass-commit mailing list