Skip to content

Extend docs by available metrics#322

Open
KonradBreitsprecherBkd wants to merge 5 commits into
mainfrom
docs/document-available-metrics
Open

Extend docs by available metrics#322
KonradBreitsprecherBkd wants to merge 5 commits into
mainfrom
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.
  • Add ATTRIBUTE as metric kind in output
  • quotes around metric value
  • add test for kind::ATTRIBUTE]
  • Fix uninitialized vars failing in mingw-release

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.

… output

Signed-off-by: Konrad Breitsprecher <Konrad.Breitsprecher@vector.com>
Signed-off-by: Konrad Breitsprecher <Konrad.Breitsprecher@vector.com>
… mingw-release

Signed-off-by: Konrad Breitsprecher <Konrad.Breitsprecher@vector.com>
Signed-off-by: Konrad Breitsprecher <Konrad.Breitsprecher@vector.com>
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