Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Output File #157

Merged
merged 4 commits into from
Dec 6, 2024
Merged

Output File #157

merged 4 commits into from
Dec 6, 2024

Conversation

pallago
Copy link
Contributor

@pallago pallago commented Nov 16, 2024

A new configuration section is introduced to write some values to a text file. This might be useful for some debugging.

Another useful application might be for instance a WebCam stream, which allows to read a text file and annotate the values into the live stream. Hence, it might be useful that some parameters from the Rpi will be also written in the annotation.

With these changes a text file can be created and the content will be also defined in the config.py

Please check (and copy) also the changed wiki
https://github.com/pallago/rpi-mqtt-monitorwiki/blob/master/Manual-Installation.md

@pallago pallago closed this Nov 16, 2024
@pallago pallago reopened this Nov 16, 2024
@pallago
Copy link
Contributor Author

pallago commented Nov 27, 2024

@hjelev ? Any changes you want, recommend, deny?

@hjelev
Copy link
Owner

hjelev commented Dec 4, 2024

Hi I like the idea, but I reworked the script a little to get rid of some old crappy code.
Can you adopt your PR to the new changes and I'll merge it.
Thanks

@hjelev
Copy link
Owner

hjelev commented Dec 5, 2024

Thanks looks good - will test it and merge during the weekend

@hjelev hjelev merged commit a4745ab into hjelev:master Dec 6, 2024
@pallago pallago deleted the write_to_text branch December 16, 2024 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants