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

Add experimental extension for metric tracer #291

Closed
wants to merge 6 commits into from

Conversation

matcabral
Copy link
Contributor

No description provided.

@matcabral
Copy link
Contributor Author

implements #287 and #285

@wdamon-intel
Copy link
Contributor

@joshuaranjan please review

scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
scripts/tools/EXT_Exp_MetricTracer.rst Outdated Show resolved Hide resolved
scripts/tools/EXT_Exp_MetricTracer.rst Outdated Show resolved Hide resolved
scripts/tools/EXT_Exp_MetricTracer.rst Outdated Show resolved Hide resolved
scripts/tools/EXT_Exp_MetricTracer.rst Outdated Show resolved Hide resolved
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
scripts/tools/metricTracer.yml Show resolved Hide resolved
scripts/tools/metricTracer.yml Show resolved Hide resolved
matcabral and others added 2 commits March 15, 2024 16:27
* Fix scripts to support two digits minor version
* Extend `all_versions` array to include 1.8-1.10.
* Add `ignore_git_revision` flag to run script.

Signed-off-by: Matias Cabral <[email protected]>
Co-authored-by: Will Damon <[email protected]>
@matcabral matcabral changed the base branch from master to staging/v1.10 March 15, 2024 20:33
@matcabral
Copy link
Contributor Author

Force push after rebasing for scripts updates

Copy link
Contributor

@joshuaranjan joshuaranjan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@matcabral
Copy link
Contributor Author

@wdamon-intel

scripts/tools/EXT_Exp_MetricTracer.rst Show resolved Hide resolved
scripts/tools/metric.yml Show resolved Hide resolved
scripts/tools/EXT_Exp_MetricTracer.rst Show resolved Hide resolved
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
@matcabral
Copy link
Contributor Author

@aravindksg, @joshuaranjan , @joshuaranjan Added a new patch to extend the metric event entry to support subdeviceId. This follows the same approach sysman already has in place.

Copy link
Contributor

@aravindksg aravindksg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

subdevice changes lgtm

@matcabral
Copy link
Contributor Author

abandoning, this was already merged and available in the spec

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API: Tools enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

Support decoding L0 Tracer Metrics Support for L0 Tracer Metrics
4 participants