Closed Aietes closed 1 year ago
Hi, yes, I already fixed that, just forgot to push it. If you pull the latest master now that error should be gone.
btw, that error should be happening only when thinkfan shuts down due to an unavailable fan. During normal operation it shouldn't be triggered. In what situation did you get it?
I got the error on starting thinkfan, actually it throwed the exception right away. I'll try the new version now.
No luck, I still get the same error on startup. Latest version, fresh build. It seems thinkfan can't find the config. Did the default location and/or filename change? I simply kept the old config with the same name in the same place, /etc/thinkfan.conf
Got it working, default path for config file seems to have changed, it's now /etc/thinkfan.yaml
Thanks for reporting back, you should definitely not be getting a std::bad_optional_access
if your config is at /etc/thinkfan.conf
, so that would be a bug.
Several issues have been fixed now that were causing a bad_optional_access
around tpacpi
sensor initialization. Assuming this was one of them. Feel free to reopen if the problem reappears with the latest version.
Hi there,
I set up a new machine, and tried to install thinkfan. Building and installing with the latest version 1.3.1 resulted in the following exception on start:
Building and running 1.2.2 works fine. Did something change between versions that I missed, e.g. different config format?