Update Alert Notebook - #129
Conversation
|
please also double check the German translation |
|
There is still quite a lot of code in the notebook, which would be better moved to the lib. Also we should aim for having only a single library (notebooks) that the users need to worry about, not 3. |
Translation LGTM |
There is some code that will be deleted once this PR is merged frequenz-floss/frequenz-reporting-python#58 I will review other parts of the notebook to see what can be moved to the library. Do you see specific parts that should be moved already?
You are right. Just realised that |
|
From my side it's fine to do this later. We should aim for notebooks that have just couple of lines for interaction with the user and display, and everything else in a python module. |
428cccd to
7d87593
Compare
- Update the introduction in the instructions. - Utilise the updated `MicrogridConfig` class to simplify the toml file loading. - Utilise the new `state_analysis` module for detecting and analysing component state transitions and alerts from reporting data. This simplifies the notebook by removing a now obsolete code. Signed-off-by: cyiallou - Costas <42914163+cyiallou@users.noreply.github.com> s
7d87593 to
e68922d
Compare
|
Updated @cwasicki |
MicrogridConfigclass to simplify the tomlfile loading.
state_analysismodule for detecting and analysingcomponent state transitions and alerts from reporting data. This
simplifies the notebook by removing a now obsolete code.