Greg Young - Help with sensESP code ..please 🙂...
Greg Young - sensESP V3 decided to give V3 an...
Matti Airas - @rszemeti : Yeah, PR would be gre...
Greg Young - trying to update older but working...
Matti Airas - And here comes!I just released ...
Configurable
and that had a non-empty config_path
were rendered as a config card. This approach mixed the embedded class implementation with presentation issues and made things like custom configuration schema more complex than necessary. In the latest version, Configurable
has been replaced with a ConfigItem
factory that needs to be called for every item you want on display. See for example the constant_sensor.cpp
example for a practical example....Matti Airas - Upcoming changes to SensESP: Refa...
Configurable
and having a config path defined would expose a REST API endpoint for GETting and PUTting its configuration, and if it has a config schema defined, a config card would also be rendered in the UI.
This approach has the problem that the configuration UI definition is tightly tied to the object, breaking the single responsibility principle and making some things more messy than necessary. I made a PR for refactoring this: now, all config items need to be defined explicitly by calling ConfigItem(obj)
. The returned pointer can be used to chain other settings together:
```c++...FlyingK - Got my test setup up and running. Tha...
FlyingK - I am struggling with the ObservableVa...
FlyingK - I had some time to start testing Sens...
./run install-frontend
to get the frontend working.
cd frontend-plugins/sample-plugin-js
pnpm install
pnpm run build
cd frontend-plugins/sample-plugin-js
pnpm install
pnpm run build
Matti Airas - Hey all, sorry about my relative ...
Configurable
class. This should result in one more beta version before the final release....barnaclebill - Has anyone found an accurate com...
MattNJ - hello, new to this today, steep learni...
Tony - Does anyone have sample code to sense vo...
Matti Airas - User Offspring (on GitHub, don) h...
clang-tidy
static analysis tool against SensESP proper and fixed a ton of warnings. The result is in this PR: https://github.com/SignalK/SensESP/pull/737
The PR does break some old code because the reactesp
namespace needs to be explicitly declared whenever the code is not using reactesp
(which is a no-no in header files)....MV 2 Umbrellas - anyone know if the sensesp lib...
MV 2 Umbrellas - I have been poking around the ...
on_receive_put
method that looks promising
https://github.com/SignalK/SensESP/blob/887f1281a8f3b91684ab96faf42b4b6546149137/src/sensesp/signalk/signalk_ws_client.cpp#L318C18-L318C32
and also this on_receive_updates
method
https://github.com/SignalK/SensESP/blob/887f1281a8f3b91684ab96faf42b4b6546149137/src/sensesp/signalk/signalk_ws_client.cpp#L248...MV 2 Umbrellas - hello folks ... I am working o...
Matti Airas - Hi all,I have now merged the de...
dev-3
development branch into main
and released version 3.0.0-beta.1. This is a major new version. While documentation hasn't been updated yet, I have listed the major changes (there are many!) and written some migration information in the repo README. I will also make corresponding update releases in related repos such as project templates etc.
Please have a go at it and report any issues and annoyances!...DWinchester - Sill looking for example with ADS...
DWinchester - Can someone point me to an exampl...