cyberman54 / ESP32-Paxcounter

Wifi & BLE driven passenger flow metering with cheap ESP32 boards
https://cyberman54.github.io/ESP32-Paxcounter/
Other
1.69k stars 397 forks source link

New the things stack v3 supports only payload decoders up to 4k (via webinterface) #804

Closed Drasek closed 2 years ago

Drasek commented 3 years ago

If you try to add the v3 payload decoder to a new application in the things stack v3 from the things network, you will get a error:

"Invalid value of field default_formatters.up_formatter_parameter"

This is the corresponding issue on the things v3 stack repo: https://github.com/TheThingsNetwork/lorawan-stack/issues/4053

If you use the API you can get it to accept up to 16k (which would be fine for us, the decoder is around 10k) and if you already got it in use you can keep it, its just about new applications/decoders.

2 possible proposals:

cyberman54 commented 3 years ago

Thanks for reporting this issue with TTN v3.

Let us go for the option to integrate paxcounter in the TTN device repo! Two thoughts:

Please note that i'm going to release v3.0.0 soon, from current code in development branch. There will be some major changes, e.g. remove wifi/ble sniffing logic from paxcounter repo, code has been moved to a new external "libpax" lib, which shall grow independently from paxcounter. And the deprecated jfrog bintray OTA update service has been exchanged by new homebrewn pax.express service.

Drasek commented 3 years ago

I like the idea about the "community", "open source", whatever vendor name, this would open up a path for other projects as well. Will the paxcounter keep its name in v3.0.0? Or will it change to ESP32-LoRa-Node / ESP-Sensor-Device or something? Would be good to use the repo name for the device name in the new device-repo just to make it clear for users which device to choose.

cyberman54 commented 3 years ago

Reponame is not planned to change.

bsiege commented 3 years ago

It seems API is now restricted at the same level...???

--default-formatters.up-formatter-parameter-local-file "Downloads/packed_decodeUplink.js" error:pkg/applicationserver:field_value (invalid value of fielddefault_formatters.up_formatter_parameter) field=default_formatters.up_formatter_parameter correlation_id=5670c43bf58243f283ef23755722cf62 --- error:pkg/applicationserver:formatter_script_too_large (formatter script size exceeds maximum allowed size) size=9858 max_size=4096 Paxcounter was my first test to migrate an application to v3. Failed stylish. Again.

Drasek commented 3 years ago

Added a pullrequest to solve this issue, with a first possible release for the lorawan-device repo from tti.

815

nottheedge commented 3 years ago

any solution available?

Drasek commented 2 years ago

This can be closed i think, after the ESP32-Paxcounter is in the lorawan-device repo and now (v3.14.0) in the things stack community edition.