Repository navigation
Alert charts, stricter UV/particulates/crossing triggers - #42
Merged
Merged
Conversation
The alert goes out as the chart with its text as the caption (and the usual buttons); a chart that cannot be made, or a picture Telegram refuses, falls back to the text alone. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WBwHofjb1g9onaYRdLZMbf
…hart to the particulates alerts One helper (lib/alerts/pictures.py) makes the chart a typed request for it would give; the crossing alert uses it too. A chart that cannot be made leaves the alert as text. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WBwHofjb1g9onaYRdLZMbf
… all three lines Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WBwHofjb1g9onaYRdLZMbf
The fake station now reports a UV index (dark at night, a daytime hump), so the UV chart is tested against real-looking data. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WBwHofjb1g9onaYRdLZMbf
…t legend: UV index, not Solar and uvi) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WBwHofjb1g9onaYRdLZMbf
…ule); temperature crossing needs more than 1 degree lib/alerts/confirm.py: a condition must hold for two readings in a row, counted by the reading's own timestamp, so a check that sees the same reading again counts for nothing. The run is saved, so it survives checks and restarts. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WBwHofjb1g9onaYRdLZMbf
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WBwHofjb1g9onaYRdLZMbf
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
2 participants
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.
Alerts carry their 1D chart
lib/alerts/pictures.py) builds each chart the way a typed request for it would; the alert is the picture with its text as the caption and the usual buttonsStricter triggers
lib/alerts/confirm.py, counted by the reading's own timestamp, saved across checks and restarts)Logs / tests
No migration needed beyond what #41 already asked for.
🤖 Generated with Claude Code
https://claude.ai/code/session_01WBwHofjb1g9onaYRdLZMbf
Generated by Claude Code