Sensors config not accepted

[backupandforkacc]

I am running the script and get the following output:

python3 app.py
Running with sensors config:
{}
nct6793-isa-0290 at ISA adapter
acpitz-acpi-0 at ACPI interface
coretemp-isa-0000 at ISA adapter
MQTT client connected with code 0
nct6793-isa-0290 at ISA adapter
acpitz-acpi-0 at ACPI interface
coretemp-isa-0000 at ISA adapter

MQTT is connected, I also send a test message. But the sensors config is probably not accepted. I made a file and called it Sensors. I took the sensor name from the output (see above), as I do not know what to use here. It has this content:

{
  "<coretemp-isa-0000>": {
    "features": {
      "<coretemp-isa-0000>": {
        "label": "Test",
        "mqtt": "coretemp-isa-0000"
      }
    }
  }
}

Here is my filestructure:

SensorsReport$ ls -lh
insgesamt 43K
-rw-rw-r-- 1 simon simon 2,5K Apr 16 12:58 app.py
-rw-rw-r-- 1 simon simon  299 Apr 15 09:49 docker-compose.yml
-rw-rw-r-- 1 simon simon  306 Apr 15 09:49 Dockerfile
-rw-rw-r-- 1 simon simon   32 Apr 15 09:49 dotenv.template
-rw-rw-r-- 1 simon simon 1,1K Apr 15 09:49 LICENSE
-rw-rw-r-- 1 simon simon 1,3K Apr 16 12:54 mqtt.py
drwxrwxr-x 2 simon simon    4 Apr 16 12:54 __pycache__
-rw-rw-r-- 1 simon simon  740 Apr 15 09:49 README.md
-rw-rw-r-- 1 simon simon   33 Apr 15 09:49 requirements.txt
-rw-rw-r-- 1 simon simon  159 Apr 16 13:11 SENSORS
-rw-rw-r-- 1 simon simon  135 Apr 15 09:49 util.py

I am just running the python scripts, no container. I adapted the mqtt.config so I do not use env variables, but direclty wrote the configuration in the files.

[penguineer]

I made assumptions about understanding of the syntax in my example, which I hopefully fixed in #2.

Please try the following configuration:

{
  "coretemp-isa-0000": {
    "features": {
      "coretemp-isa-0000": {
        "label": "Test",
        "mqtt": "coretemp-isa-0000"
      }
    }
  }
}

Can you please check your "feature" entry? For me, this is temp one and I have written my configuration like this:

{
  "coretemp-isa-0000": {
    "features": {
      "temp1": {
        "label": "CPU Package",
        "mqtt": "CPU/Package"
      }
    }
  }
}

[penguineer]

I am just running the python scripts, no container.

For a controlled environment I propose you use the docker image (see additions to README on how to run from the pre-built image) and create a .env file from the dotenv.template.

I adapted the mqtt.config so I do not use env variables, but direclty wrote the configuration in the files.

The application is expecting the sensors configuration in an environment variable. If you want to load from a file, changes to the source code are needed. Naming the file SENSORS will not make it available in the environment.

Is there a reason why you do not want to go with the configuration structure proposed in this project?

[penguineer]

I have just released v1.1.0, which adds configuration validation and also issues a warning if no sensors have been found in the configuration (see #7 and #14).

Please re-open if there are still issues with the configuration.