-
Notifications
You must be signed in to change notification settings - Fork 582
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
Fix critical issue with HUMID module #1857
Conversation
Fix issue-1856, which was caused by the fact that the process_stats function did not return anything on success, causing the data to be discarded.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Module needs a bit more refactoring to clean up functions outside object.
I can try to have a go at the changes later if I get a moment (and if you don't beat me to it 👍🏻 ) |
I've refactored the code a bit more, hopefully it is clearer now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did my own minor refactoring, hope that's ok... Mostly a personal taste thing, but I find this a little less complex (no copying / passing dicts around) and easier to read / maintain.
Thanks for the fix! ✨
See #1856 for details.