Skip to content

Extend docs by available metrics#322

Open
KonradBreitsprecherBkd wants to merge 1 commit intomainfrom
docs/document-available-metrics
Open

Extend docs by available metrics#322
KonradBreitsprecherBkd wants to merge 1 commit intomainfrom
docs/document-available-metrics

Conversation

@KonradBreitsprecherBkd
Copy link
Copy Markdown
Contributor

@KonradBreitsprecherBkd KonradBreitsprecherBkd commented May 7, 2026

Add docs for available metrics and some example output.
Partly generated with GPT-5.
https://github.com/vectorgrp/sil-kit-planning/issues/46

Signed-off-by: Konrad Breitsprecher <Konrad.Breitsprecher@vector.com>
@KonradBreitsprecherBkd
Copy link
Copy Markdown
Contributor Author

@VDanielEdwards
Mir ist noch aufgefallen, dass für Metric Kind ATTRIBUTE, im Json eine "3" landet, und bei den werten für ATTRIBUTE fehlen noch quotes für valides json:

{"ts":93616265196100,"pn":"CanReader","mn":"SilKit/System/ProcessorArchitecture","mk":"3","mv":x86_64}

Ist das durchgerutscht oder Absicht?

@VDanielEdwards
Copy link
Copy Markdown
Member

VDanielEdwards commented May 8, 2026

@VDanielEdwards Mir ist noch aufgefallen, dass für Metric Kind ATTRIBUTE, im Json eine "3" landet, und bei den werten für ATTRIBUTE fehlen noch quotes für valides json:

{"ts":93616265196100,"pn":"CanReader","mn":"SilKit/System/ProcessorArchitecture","mk":"3","mv":x86_64}

Ist das durchgerutscht oder Absicht?

That was not done deliberately, but likely just forgotten. The enumerator / string must be added here.

For the missing quotes, the the code should wrap the field for output in SilKit::Util::EscapedJsonString here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants