DigitalShoestringSolutions / PowerMonitoring

GNU General Public License v3.0
0 stars 1 forks source link

Update pm_a_3p.toml #16

Closed Greg-Hawkridge closed 1 month ago

Greg-Hawkridge commented 2 months ago

Addresses #13 to meet new config format

Greg-Hawkridge commented 1 month ago

@tobyaharris All working now

tobyaharris commented 1 month ago

@Greg-Hawkridge which hardware did you test it on?

(that is to cheekily say, the only evidence I heed that some code works is the results of a test on hardware)

tobyaharris commented 1 month ago

Of course I've since tested this branch on Table 1. Build time 553s. The sensing module seems happy running in advanced 3ph mode, but the pi is running so slowly that in half an hour it hasn't managed to cough up the docker logs.

However I must add as I tested on a 2gb pi (vcgencmd get_config total_mem) the Grafana and Influx web interfaces were totally unusuable. SSH keeps timing out. The grafana dashboard has been loading for over half an hour and still hasn't got a single data point. See https://github.com/DigitalShoestringSolutions/PowerMonitoring/issues/17.

tobyaharris commented 1 month ago

Testing what happens if all module config files are commented out:

current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
current-sensing-1  | INFO:config:Loaded config file "./config/user_config.toml" specified in default
current-sensing-1  | CRITICAL:config:Config File Not Found - unable to load config file "./config/module_config.toml" specified by default.
Greg-Hawkridge commented 1 month ago

@tobyaharris should be ready now