Add Streamlit interface for motion mosaics#282
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
| mosaic_path = create_mosaic(images, OUTPUT_MOSAIC, cols=8, rows=4) | ||
|
|
||
| send_via_telegram([mosaic_path, PEOPLE_GRAPH, DISK_GRAPH]) |
There was a problem hiding this comment.
Handle empty motion directory before generating mosaic
In the default code path the script calls latest_images(32) and immediately passes the result into create_mosaic. When /var/lib/motion has no recent files (common on first boot or if Motion stopped) latest_images returns an empty list and create_mosaic raises a ValueError, terminating the run before any telemetry is sent. The --date branch already handles this case explicitly; the live branch should similarly bail out or show a warning instead of crashing.
Useful? React with 👍 / 👎.
Summary
Testing
https://chatgpt.com/codex/tasks/task_e_68e9c8a55b288331a7b4c350eee78eea