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

global.json and <color>.json written to SD every 10 seconds #9

Open
fgleason opened this issue Jan 11, 2019 · 4 comments
Open

global.json and <color>.json written to SD every 10 seconds #9

fgleason opened this issue Jan 11, 2019 · 4 comments

Comments

@fgleason
Copy link

Could those files be disabled by default? They appear to be unnecessary SD writes. If they are required could you write to /tmp and use tmpfs for /tmp?

@noahbaron
Copy link
Collaborator

Thanks for your feedback. Those writes are for restoring the system during a power outage or reboot. I understand there is a disadvantage to excessive writes to the SD card. If there is a more reliable memory location to write that can be accessed between reboots that would be ideal. If you have any references for tmpfs, let me know and will be happy to change.

@fgleason
Copy link
Author

Hmm that is a challenge. Do you need the current values to restore after reboot or just the configuration information? If it's just the configuration it really only needs to be written when it changes. Perhaps nice to have but not required to restart information could be disabled by the user in the GUI if they want to. Or maybe write less frequently. It does not sound like tmpfs would work for this. Here is a reference: http://man7.org/linux/man-pages/man5/tmpfs.5.html

@noahbaron
Copy link
Collaborator

Just the configuration information, basically anything the user inputted like preferred units, calibration, beer name, cloud logging id, and custom cloud URLs. I could reduce the writes by only writing if the configuration data changes and not if the values like SG or temperature change, which would be a substantial reduction

@fgleason
Copy link
Author

That would be great. Thanks very much.

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

No branches or pull requests

2 participants