Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kapsch ETX S0021 s should have a character for each detector logic #518

Open
emiltin opened this issue Jan 31, 2025 · 1 comment
Open

Kapsch ETX S0021 s should have a character for each detector logic #518

emiltin opened this issue Jan 31, 2025 · 1 comment
Labels

Comments

@emiltin
Copy link
Contributor

emiltin commented Jan 31, 2025

The ETX sends "-" in S0021, even though the config lists several detector logics. The s string should contain one character per detector logic.

    2025-01-30T22:49:46.232Z  52068  KTT+TC=101TC001      KTT+TC=101TC001      1263  Received invalid StatusResponse schema errors: /sS/0/s, pattern,  {"mType":"rSMsg","type":"StatusResponse","mId":"1263e11c-06c0-44fd-814e-45f8f9a70f0a","sTs":"2025-01-30T22:49:45.431Z","ntsOId":"","xNId":"","cId":"KTT+TC=101TC001","sS":[{"sCI":"S0021","n":"detectorlogics","q":"recent","s":"-"}]}

Note: The string "-" is currently rejected due to a schema error, but this is due to an omission in the schema. It's currently not allowing dashes in the regex pattern, but it should:
rsmp-nordic/rsmp_schema#127

@emiltin emiltin added the ETX label Jan 31, 2025
@emiltin
Copy link
Contributor Author

emiltin commented Jan 31, 2025

I think the same is the case for S0002, causing the test "Site::Traffic Light Controller Detector Logic forcing is set with M0008" fails:
https://github.com/rsmp-nordic/rsmp_validator/actions/runs/13062231280/job/36447666042#step:6:596

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant