-
Notifications
You must be signed in to change notification settings - Fork 42
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
add more and improve diagnostic #467
Comments
TriceOverflowCounter is ok, see also #462. I cannot understand the value of the proposed TriceWordCounter. Please give an example. |
TriceWordCounter, reseting it to 0 on very diagnostic update, it would be useful to better understand the amount of data of log produced during that time window. (maybe this could be also get indirectly by other statistics) |
Maybe you mean the used max depth of a double half buffer or the max depth of the ring buffer with the expression "TriceWordCounter"? That is a very important metric and available when enabling TRICE_DIAGNOSTICS. |
I'm thinking / trying to remember .. :)
What I mean is, the "max depth" only mean that (as the depth can change between diagnostic reports), TriceWordCounter would mean the total "traffic" that happened during the diagnostic report. So "max depth" != TriceWordCounter Eg: |
Not really convinced, so far. A traffic measurement could be done by the Trice tool and should not be a task of the embedded device. Also it is not clear what to measure: The Trice netto data? The framed data, with or without 0-delimiters? Simply the Trice count? We could extend the trice tool for that. The easiest way would be to add it to the statistics when shutting down the tool. |
In main branch now is a separate file |
Protect error count:
Add overflow counter:
Add word counter so we know / can verify the number of words inserted (it can be reset per diagnostic update)
The text was updated successfully, but these errors were encountered: