Add support for sensors from liquidctl #202
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Requires liquidctl v1.7.0 or later, or a current git snapshot.
|
liquidctl 1.7.0 has been released with the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
liquidctl is a cross-platform tool to monitor and control many all-in-one liquid coolers, fan controllers, PSUs and some other devices. This PR adds support for showing sensor data from liquidctl in Freon.
Since many types of sensors are supported, merge the data from
liquidctlandsensorsand use the generic temperature, rpm and voltage sections. Also add a new switch to the preferences to enable/disable this feature.This PR uses a new
--jsonflag from liquidctl, and this currently requires a recent git snapshot. Because of this, keep it as a draft PR for now, but those comfortable with git can already start testing/using it.Any feedback would be very appreciated.