Regenerate toy tutorial screenshots for kW units#15
Open
Flix6x wants to merge 3 commits into
Open
Conversation
The toy tutorial moves from MW-scale to kW-scale power sensors and from EUR/MWh to EUR/kWh day-ahead prices (FlexMeasures/flexmeasures#2223), so every chart in the tutorial showed stale axis labels and values. Regenerated by running all five toy tutorial parts against a fresh database on that branch and exporting each chart with the UI's own "Save as PNG" action. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The toy building's consumption-price sensor now reports EUR/kWh instead of EUR/MWh, which the flex-context editor shows. The other toy tutorial images were re-rendered too, but left as they are: the battery flex-model (450 kWh / 500 kVA / 90%) and the site structure are unaffected by the unit change, and the dashboard and building status pages could not be reproduced faithfully outside a full tutorial stack (no Mapbox tiles, no Redis). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Retake the site structure and battery flex-model screenshots so the browser URLs show the current toy asset IDs used by the tutorial: /assets/5/context and /assets/6/properties.
Contributor
|
I verified the screenshots. They look good overall; I only found two screenshots with stale asset IDs in the browser URL. I refreshed those so the site structure screenshot now shows |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Companion to FlexMeasures/flexmeasures#2223, which switches the toy tutorial to kW-scale power sensors and EUR/kWh day-ahead prices. That PR's "Further Improvements" asks for the external UI screenshots to be regenerated here.
How these were made
All five toy tutorial parts were run in order against a fresh database on the
feat/change-toy-tutorial-to-kw-unitsbranch, and each chart was exported with the chart's own Save as PNG action in the UI (not a browser screenshot), so the framing matches how the existing charts were produced.The run reproduces the numbers the tutorial docs claim on that branch: headroom bottoms out at 280 kW around noon, and the process costs come to 9.60 / 4.30 / 5.40 EUR for the inflexible / breakable / shiftable policies.
Commit 1 — charts affected by the unit change
Power axes go from
MW(±0.5) tokW(±500), prices fromEUR/MWh(4–18) toEUR/kWh(0.004–0.018), and the process cost reports from hundreds of EUR to fractions of a EUR:asset-view-process,asset-view-with-solar,asset-view-without-solar,sensor-data-breakable,sensor-data-charging,sensor-data-charging-with-solar,sensor-data-headroom,sensor-data-headroom-nopv,sensor-data-headroom-pv,sensor-data-inflexible,sensor-data-multiasset-negativeprices,sensor-data-prices,sensor-data-production,sensor-data-shiftable.Two incidental differences from the old set: the charts now carry the plot titles the UI renders today ("Prices", "Power flows"), and the asset-page charts include the price panel that the old images were cropped to exclude.
Commit 2 — the other toy tutorial images
Only
toy-tutorial-building-flex-context.pngchanges: the consumption-price sensor now readsUnit: EUR/kWh.The rest were re-rendered and deliberately left alone:
toy-tutorial-battery-flex-modeltoy-tutorial-site-structureasset-view-dashboardscreenshot_building_statusNote that all four (plus the flex-model one) are also stale in other ways relative to the current UI — the starter template assets now shift the toy sensor IDs, so the IDs in the images and in the tutorial docs no longer line up. That's worth a separate pass and is not caused by #2223.