Commit Graph

58 Commits

Author SHA1 Message Date
Pi Developer 9d6e3b31f0 Usa GITEA_TOKEN secret per autenticazione Container Registry
Build Docker Image for Raspberry Pi / build-and-push (push) Failing after 34s
- Modificato docker-build.yml e docker-cleanup.yml
- Il token automatico github.token non ha permessi sufficienti
- Aggiunta guida SETUP_GITEA_TOKEN.md per configurazione
2026-01-25 15:33:17 +01:00
Pi Developer bfc7c21559 Aggiungi permessi packages per accesso al Container Registry
Build Docker Image for Raspberry Pi / build-and-push (push) Failing after 36s
- Specificato permissions: packages: write nei workflow
- Necessario per autenticazione al registry Gitea self-hosted
2026-01-25 15:16:52 +01:00
Pi Developer 6d4812efbc Correggi workflow Gitea Actions per usare registry self-hosted
Build Docker Image for Raspberry Pi / build-and-push (push) Failing after 39s
- Modificato docker-build.yml per usare gitea.home-nas-ds.org invece di Docker Hub
- Modificato docker-cleanup.yml per autenticazione con registry Gitea
- Usato github.token automatico invece di secrets esterni
- Aggiornata documentazione per riflettere i cambiamenti
2026-01-25 15:15:17 +01:00
Pi Developer 6af9fcad7e feat(v2.0): Auto-discovery, correzione bug critici, e documentazione completa
Build Docker Image for Raspberry Pi / build-and-push (push) Failing after 4m42s
🆕 Funzionalità Auto-Discovery
- Aggiunto metodo AutoDiscoverBufferSizes() per rilevamento automatico QPIGS/QPIRI/QMOD/QPIWS
- Supporto variabili d'ambiente (INVERTER_DEVICE, MQTT_SERVER, etc.)
- Caching persistente buffer sizes in /cache/inverter.conf.cache
- Flag -a/--auto-discover per modalità auto-detection

🐛 Bug Fixes Critici
- **Parsing interi**: Aggiunta attemptAddSettingInt() con stoi() invece di stof()
  - Fix: stof('98') = 98.0f → 97 (int), ora stoi('98') = 98 direttamente
  - Applicato a: qpiri, qpiws, qmod, qpigs
- **Thread sync**: Aggiunto ups_qpiws_changed a main loop e condizione exit poll()
  - Fix: loop principale controllava solo 3 flag su 4, causava hang
  - Fix: thread poll() non usciva in runOnce perché mancava controllo QPIWS
- **Config accuracy**: Corretti buffer sizes (qpiri: 98→103, qpiws: 36→40)
  - Rimosso sources/inverter-cli/inverter.conf che sovrascriveva config globale
  - Validato con test: inverter_poller -1 completa in 6s con JSON completo

📚 Documentazione Completa
- Creato documentation/CODE_ARCHITECTURE.md (38KB)
  - Mappa logica variabili globali
  - Flusso esecuzione main() con diagrammi ASCII
  - Sequence diagram classe cInverter (poll, query, auto-discovery)
  - Thread synchronization diagrams
  - MQTT integration bash scripts flow
  - Mappa concettuale 5-layer system architecture
  - Error handling e performance optimizations
- Organizzati file .md in documentation/ (AUTO_DISCOVERY, IMPLEMENTATION, QUICKSTART, DEBUG)
- Aggiornato README.md con sezione v2.0 e indice documentazione
- Aggiornato .github/copilot-instructions.md con novità v2.0

🔧 Miglioramenti Build & CI/CD
- Gitea Actions per build multi-arch (arm/v6, arm/v7, arm64, amd64, 386)
- Configurazione VS Code completa (tasks, launch, debug GDB)
- Script test-autodiscovery.sh e test-device.sh

 Testing Validato
- inverter_poller -1 completa in 6 secondi
- Output JSON completo con tutte le metriche
- Exit pulito senza timeout (exit code 0)
- Tutte le 4 query QMOD/QPIGS/QPIRI/QPIWS funzionanti
2026-01-25 15:00:48 +01:00
David Nedved ac2642639f Merge pull request #108 from tititg34/patch-2
Update mqtt-init.sh
2025-06-19 08:58:42 +10:00
tititg34 51b6a65973 Update mqtt-init.sh
temperature unit
2023-09-07 18:16:25 +02:00
David Nedved a96592ad21 Merge pull request #70 from dilyanpalauzov/main_cpp_tautology
inverter-cli/main.cpp: remove tautology
2022-08-25 14:09:08 +10:00
Дилян Палаузов 730770016c inverter-cli/main.cpp: remove tautology
Closes: https://github.com/ned-kelly/docker-voltronic-homeassistant/issues/68
Related-To: https://github.com/manio/skymax-demo/issues/10
2022-03-13 11:57:19 +02:00
ned-kelly 0ddffda79d Merge pull request #45 from RSd108/master
Include a clientid in mqtt commands
2021-10-06 22:36:21 +10:00
RSd 037ce5f37b Make the clientid configurable 2021-06-03 14:47:59 +02:00
RSd108 5ba1072099 Use client identifier 2021-06-03 12:47:37 +02:00
RSd108 a3957d0326 Use client identifier 2021-06-03 12:46:01 +02:00
RSd108 9d646bad60 Use client identifier 2021-06-03 12:44:58 +02:00
ned-kelly 24460639ac Merge pull request #35 from therokh/therokh-fix-influx-enabled-check
Fix IF check space issue for INFLUX_ENABLED
2021-01-18 09:18:13 +10:00
Sam e82000d76f Fix IF check space issue for INFLUX_ENABLED 2020-12-03 18:14:42 +10:30
ned-kelly d8349b8574 Update mqtt.json 2020-11-23 08:48:14 +10:00
ned-kelly f19040ab50 Merge pull request #32 from marval/write-to-influx
Added the possibility to write directly to Influx
2020-11-23 08:47:21 +10:00
Martin Valov 515d8196f6 Added the enabled true|false flag in mqtt.json 2020-11-22 21:53:40 +01:00
Martin Valov 313c98c119 Added the possibility to write directly to Influx
Since I am not using Influx as a DB of my HomeAssistant setup, I wanted a way to bring the data there. Another issue is that I used another script to gather the data and hence have other naming conventions on my dashboards. Therefore the name of the measurements are now part of the config JSON file. This could be an idea adopted to MQTT as well.
2020-11-08 22:38:50 +01:00
David a7f161b9d6 update readme 2020-09-19 12:42:49 +10:00
David Nedved 774606e5e6 include grafana dashboard sample 2020-05-04 10:05:53 +10:00
David Nedved 9b49015fd7 docker compose to default to base image 2020-04-07 13:54:33 +10:00
David Nedved 6014f4f2b1 Merge branch 'actions' 2020-04-01 12:06:08 +10:00
David Nedved e517950a87 update readme 2020-04-01 12:06:00 +10:00
ned-kelly faca12b5cd Merge pull request #18 from ned-kelly/actions
Merge automated CI/CD Workflow for multi-arch builds into Prod
2020-04-01 12:02:22 +10:00
David Nedved b8e9d8ad33 include healthcheck in default actions 2020-04-01 11:42:35 +10:00
David Nedved a30d4019fd add docker image name as environment 2020-04-01 10:47:05 +10:00
David Nedved 0ea9fa29b6 test with lest arch options 2020-04-01 10:43:49 +10:00
David Nedved 2aa7edd04b remove line break slash from tags in dockerfile 2020-04-01 09:40:40 +10:00
David Nedved 1d87c1cc6d restructure dockerfile 2020-04-01 09:39:29 +10:00
David Nedved a83f79fc7e add actions branch triggers for testing 2020-04-01 09:37:18 +10:00
David Nedved 6000d93e84 initial support for docker actions 2020-04-01 09:33:45 +10:00
David Nedved 141829f722 initial support for docker actions 2020-04-01 09:32:19 +10:00
David Nedved d5dc992a62 remove duplicate battery voltage arg 2020-04-01 09:07:06 +10:00
ned-kelly 58ad990401 Merge pull request #14 from wax911/develop
Support for configurable parameters for inverter command response buffers
2020-03-16 13:32:27 +10:00
Maxwell f6c4ed48f5 Merge branch 'feature/buffer-configuration' into develop 2020-03-10 10:09:59 +02:00
Maxwell 6b5c5d600a Load command buffer sizes from configuration
Sets values for qpiws, qmod and qpigs from configuration file if they are available.

See commit: dfbb0ecc06
2020-03-10 10:05:45 +02:00
Maxwell de8bc8f812 Update inverter.cpp allowing the usage of configured parameters
See commit: f7896ecd7e
2020-03-10 10:02:25 +02:00
Maxwell f7896ecd7e Add additional fields to cInverter parameterized constructor
Adds the capability to use non static values for `QPIWS`, `QMOD` and `QPIGS` commands
2020-03-10 09:59:22 +02:00
Maxwell dfbb0ecc06 Add additional configurable parameters for buffers
Adds additional `qpiws`, `qmod` and `qpigs` parameters with default values used in source code, these values are currently based on the **skymax** inverter
2020-03-10 09:52:13 +02:00
Maxwell b25795841e Correct minor typo on watt_factor comment 2020-03-10 09:41:44 +02:00
ned-kelly 026786c332 Merge pull request #13 from raph2i/add_qpiri
Add qpiri as a config value
2020-02-27 15:01:47 +10:00
Raphael Pertl b39eeb52eb fix example config, cpp errors, tested with voltronic 5000 and qpiri 102 2020-02-22 23:04:00 +01:00
Raphael Pertl 35a09bb538 add qpiri as a configurable value 2020-02-22 22:21:11 +01:00
David ebd3bbe2bc include support for MQTT username/password 2019-08-12 15:17:54 +10:00
David b1399bab57 remove retain from mqtt init message 2019-08-12 12:37:40 +10:00
David 293f487ef2 update readme screenshots 2019-08-09 16:34:43 +10:00
David 0c2c945a96 update readme with more info around charging VLRA batteries 2019-08-09 16:17:37 +10:00
David 84cfa20aee fix missing paths and json formatting after refactor testing 2019-07-27 23:03:08 +10:00
David ccb999b8dc refactor for better support with ch341 and other unreliable rs232 adapters 2019-07-27 22:34:34 +10:00