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

Implemented config file #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Implemented config file #29

wants to merge 1 commit into from

Conversation

akotulu
Copy link

@akotulu akotulu commented Oct 9, 2023

  1. Plugin failed to run with older version of liquidctl cause of the StatusRecord.value has type float, but ctl returned a string. Improved the status parsing to log failed devices. Now plugin will be loaded regardless of parsing issues.
  2. Implemented config file called FanControl.Liquidctl.json. It supports sending custom match commands to device (e.g. set color).
{
  "liquidctl": {
    "exePath": "liquidctl.exe",
    "match": [
      {
        "device": "kraken",
        "set": [
          "ring color breathing 05445E 189AB4 75E6DA D4F1F4",
          "logo color spectrum-wave --speed slower"
        ]
      }
    ]
  }
}

… and improved status parsing to skip failed devices with logging.
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.

1 participant