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.

Sign up for GitHub

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

Jump to bottom

fix: allow upper case sensor names #429

Merged
merged 2 commits into from
Nov 29, 2021
Merged

fix: allow upper case sensor names #429

merged 2 commits into from
Nov 29, 2021

Conversation

vajonam
Copy link
Contributor

@vajonam vajonam commented Nov 22, 2021

when using sensor names like this

[temperature_sensor HAL]
sensor_type: temperature_host
sensor_path: /sys/devices/platform/coretemp.0/hwmon/hwmon0/temp3_input

it was causing the SystemLoads panel to break, mostly because they keys were being looked up were all lowercase. This correctly looks up the keys.

@meteyou meteyou merged commit b6cfb0f into mainsail-crew:develop Nov 29, 2021
@vajonam vajonam deleted the uppercase_sensor_names branch November 29, 2021 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@vajonam @meteyou