[Rel 7.0 RC3] [rocpd] Adding summary module to generate summaries from rocpd databa…#190
Closed
systems-assistant[bot] wants to merge 2 commits intorelease/rocm-rel-7.0from
Conversation
…se + query submodule + rocpd command-line tools (#488) * adding summary.py to generate tmp <category_region>_summary views * migrating CSV summary to SDK method of writing CSVs - Add domain_view to summary.py - omit the C++ code of writing CSV because it gets revered later anyway * Add summary subparser and write_sql_view_to_csv function * adding all <>_summary views generation to summary.py * add summary_per_rank feature * add --summary-per-rank * reconstruct generate_summary_view and create_domain_view -introduce by_rank * remove sqr and variance in summary views * use RocpdImportData instead of connection * two fixes on summary.py --modify the generate_summary_view function to return a tuple with view name and sql code add if_not_exits parameter to generete_summary_view * Refactor summary.py to allow output path and filename args, and apply time_window - clean up summary table column headers - only generate by-rank views if that param is specified * Add ProcessID to Hostname output and csv, so users can identify the system in the by-rank summaries * Summary.py, just add hostname to by-rank summaries, instead of creating mapping table * Summary - migrate csv writer to pandas, for more future flexibility * Adding a few simple tests for summary.py * Linting fixes * add region_categories to summary options - Automatically retrieve region categories from the database if argument is None * add backticks for view_names * fix tests after rebase * Made code review changes - fixed whitespace in CMakelists.txt - adding query.py module & subparser in __main__.py - refactor summary function to return query - used query.py to output csv - used query.py to also output summary to console - provided new command line options to select summary output to csv or console * Made fix to jinja template in query.py, as suggested by copilot * Consolidated output calls to query in export_view function based on feedback - refactored: helpers, query functions, create view functions - extended formats to include what query supports (md, html, pdf, json) - added json format to query, and changed orient=records - adding jinja2 and reportlab to requirements.txt * Add version_info for rocpd and roctx * Add rocpd commandline tool * Add executable permissions to source/bin/rocpd.py * Removed rocpd2query, and cleaned up --help examples --------- Co-authored-by: acanadas <acanadas@amd.com> Co-authored-by: Jin Tao <jintao12@amd.com> Co-authored-by: a-canadasruiz <Araceli.CanadasRuiz@amd.com> Co-authored-by: Jonathan R. Madsen <Jonathan.Madsen@amd.com>
…elease_rocm-rel-7.0/ROCm_rocprofiler-sdk/yhui_rocpd-summary-7.0
systems-assistant
bot
requested review from
SrirakshaNag,
bgopesh,
jrmadsen and
t-tye
as code owners
abchoudh-amd
pushed a commit
that referenced
this pull request
Aug 8, 2025
[ROCm/rocprofiler-sdk commit: 03e5a1d]
ajanicijamd
pushed a commit
to ajanicijamd/rocm-systems
that referenced
this pull request
Aug 8, 2025
Update Dyninst submodule Refactoring of build scripts to build TBB, Boost, ElfUtils, and LibIberty, since Dyninst build scripts no longer do. Workflows are now building Dyninst and its dependencies. --------- Co-authored-by: marantic-amd <marantic@amd.com> Co-authored-by: David Galiffi <David.Galiffi@amd.com> [ROCm/rocprofiler-systems commit: 96df9b6]
Contributor
|
7.0 RC, closing |
ammallya
pushed a commit
that referenced
this pull request
Jan 21, 2026
Changed the data type of `size` to `size_t` in all functional tests to ensure consistency with rocSHMEM APIs.
ammallya
pushed a commit
that referenced
this pull request
Jan 21, 2026
Changed the data type of `size` to `size_t` in all functional tests to ensure consistency with rocSHMEM APIs. [ROCm/rocshmem commit: 7a5c6f8]
ammallya
pushed a commit
that referenced
this pull request
Jan 30, 2026
* Resolve an issue for saving decoded jpeg images * update version and changelog * address review comment * clean up
ammallya
pushed a commit
that referenced
this pull request
Jan 30, 2026
* Resolve an issue for saving decoded jpeg images * update version and changelog * address review comment * clean up [ROCm/rocjpeg commit: 6ad2925]
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…se + query submodule + rocpd command-line tools (#488)
https://ontrack-internal.amd.com/browse/SWDEV-504931
PR Details
Associated Jira Ticket Number/Link
https://ontrack-internal.amd.com/browse/SWDEV-504931
What type of PR is this? (check all applicable)
Technical details
Added/updated tests?
Updated CHANGELOG?
Added/Updated documentation?
🔁 Imported from ROCm/rocprofiler-sdk#124
🧑💻 Originally authored by @rocm-devops