plotting is fragile
The idea to store all plotted variables in the module global list __plotvars
leads to problems, when SaQC is directly called (i.e. not through the CLI) several times and the configuration changes between calls.
To reproduce the issue, create a configuration with two distinct variables (e.g. var1
and var2
) and call the function saqc.core.run
, then change the configuration to a single variable (e.g. var1
) and run
again.
The __plotvars
list now holds all previously plotted variables ([var1, var2, var1]
) but during the second call, the flagger
only knows var1
and fails to access var2
.