[yew-devel] [PATCH yew-comp 00/20] refactor and improve rrd graph code

Dominik Csapak d.csapak at proxmox.com
Fri May 30 14:21:42 CEST 2025


This series aims to make the code of the rrd graphs more readable and
maintanable, by:
* moving code into it's own directory/file structure
* move code/structs together that belong together thematically
* factour out calculation code from graph component construction
* documenting relevant functions
* removing unnecessary computations

While the code now sometimes iterates over the data multiple times to
1. create the baseline data
2. create the actual components

the first part is now only done when necessary (when data/time changes,
when the user selects a new range) and not every time the graph is
redrawn, e.g. when the user moves the curser over the data.

So all in all this should do less work than before.

Also this fixes some panics when dealing with edge cases (e.g. clicking
on empty graphs, having a range selected then changing the underlying
data so that the range is not there anymore, ect.)

While at it, remove the old rrd graph code that used uplot. This is not
in use anywhere anymore but can be easily restored if necessary.

Future work could be to include more than two series. This can be done
later though, since we don't require that at the moment.

Dominik Csapak (20):
  remove old rrd uplot code
  rrd: refactor code for compute_min_max
  rrd: move into own module
  rrd: move unit calculation to own module
  rrd: units: add tests
  rrd: units: simplify calculations for get_grid_unit_base
  rrd: remove unnecessary `no_data` field
  rrd: align tooltip directly to pointer position
  rrd: use 'cross_pos' state instead of 'draw_cross'
  rrd: give all elements in svg keys
  rrd: simplify toggle Msg
  rrd: remove wrongly annotated lifetime
  rrd: refactor series related struct and functions into own module
  rrd: clamp view range when time_data changes
  rrd: refactor grid data computation
  rrd: introduce GraphSpace struct and use it to precalculate graph data
  rrd: precalculate the grid line and label positions
  rrd: calculate series svg data only when necessary
  rrd: refactor selection rectangle calculation
  rrd: refactor the cross position calculation

 js-helper-module.js    |   18 -
 src/lib.rs             |   19 +-
 src/rrd/graph.rs       |  726 ++++++++++++++++++++++++++++
 src/rrd/graph_space.rs |  147 ++++++
 src/rrd/grid.rs        |  147 ++++++
 src/rrd/mod.rs         |   11 +
 src/rrd/series.rs      |   96 ++++
 src/rrd/units.rs       |  326 +++++++++++++
 src/rrd_graph.rs       |  169 -------
 src/rrd_graph_new.rs   | 1038 ----------------------------------------
 10 files changed, 1455 insertions(+), 1242 deletions(-)
 create mode 100644 src/rrd/graph.rs
 create mode 100644 src/rrd/graph_space.rs
 create mode 100644 src/rrd/grid.rs
 create mode 100644 src/rrd/mod.rs
 create mode 100644 src/rrd/series.rs
 create mode 100644 src/rrd/units.rs
 delete mode 100644 src/rrd_graph.rs
 delete mode 100644 src/rrd_graph_new.rs

-- 
2.39.5





More information about the yew-devel mailing list