* Fix roofline rocm version bug
* Fix utils bug
* Remove unnecessary tests
* Do not check textual-fspicker package in cmake build
* Use rocprofv3 to test MI 100 and fix tests
[ROCm/rocprofiler-compute commit: 000fd4f5b2]
* Update current bins to have rocm6 suffix. Add new rocm7 bins, built on rocm7.0 latest due to hip updates.
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
* Re-add rocm version check for roof bins.
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
* Rebuild roofline binaries on top of latest rocm7 (#16379) after recent mainline promotions.
Adjusting version and distro combinations of bins following rocm6 vs rocm 7 supported OS.
*rhel8 not supported on rocm7, also not built anymore
*sles15 not supported on rocm7 but is still being built
*ubuntu stays as 22.04 and above for rocm7
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
* Minor fixes after testing.
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
* Fixed bad copy after finding it in testing ctest.
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
* Remove runpath from new bin
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
* Rework target_binary map return in detect_roofline- we should not be returning maps of different sizes or with different keys for the same method. Expected output should be consistent in case we run into bad position, or for testing purposes. Manually tested all possible roofline bin expected cases to comfirm functionality and expected user output.
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
* Update changelog with new roofline distro minimums
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
---------
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
[ROCm/rocprofiler-compute commit: 3df0d1dded]
Adjusting expected output list for test_block_SPI for MI350 series products.
Run formatter.
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
[ROCm/rocprofiler-compute commit: 61a9381edf]
Support for python3.8 and above means union defined in one of the methods was throwing errors for anything less than python3.10. Swapping out | operand for Optional[] resolves errors on systems using <3.10. No functional changes.
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
[ROCm/rocprofiler-compute commit: a59b1ea6e6]
* Add custom rocprofiler-sdk counter definitions file for MI 100
* Update CHANGELOG to mention that accumulation counters will not be
collected when profiling on MI 100 using rocprofiler-sdk/rocprofv3
* Migrate accum_counters.yaml to code
[ROCm/rocprofiler-compute commit: a95a45d69a]
Change when Roofline PDFs are generated- during general profiling and --roof-only profiling (skip only when --no-roof option is present)
---------
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
[ROCm/rocprofiler-compute commit: 630bc149ff]
* add paths-ignore for both push and PR to rhel-8.yml and ubuntu-jammy.yml
* add paths-ignore for PR only in tarball.yml
* add rocm-docs-core linting.yml workflow to formatting.yml
* fix small mistake in github directory path
* remove rocm-docs-core linting
[ROCm/rocprofiler-compute commit: 44c3516b8b]
* added progress printout for profiler
* added comments and fixed readability
* removed redundant newlines
* moved format_time helper function to utils
* removed tqdm and redundant time calc
[ROCm/rocprofiler-compute commit: 036866fc12]
* updated documentation wording, added cherry pick option for PR type
* moved pull_request_template to base .github directory
[ROCm/rocprofiler-compute commit: da6bf4b5c9]
* Fix: Add explicit includes for assert.h and thread in sample/vcopy.cpp
The HIP/CLR change e3cb5399c removed transitive inclusion
of standard headers like assert.h from hip_runtime.h.
This caused build failures in rocprof-compute samples.
This commit explicitly includes <assert.h> and <thread>
in vcopy.cpp to resolve potential missing definitions.
Signed-off-by: jamessiddeley-amd <James.Siddeley@amd.com>
* Update vsequential_access.cpp
Added assert and thread imports
Signed-off-by: jamessiddeley-amd <James.Siddeley@amd.com>
* Update vrandom_access.cpp
Added assert import
Signed-off-by: jamessiddeley-amd <James.Siddeley@amd.com>
* Update vsequential_access.cpp
Signed-off-by: jamessiddeley-amd <James.Siddeley@amd.com>
---------
Signed-off-by: jamessiddeley-amd <James.Siddeley@amd.com>
[ROCm/rocprofiler-compute commit: 667128e3c7]
* Revert of https://github.com/ROCm/rocprofiler-compute/pull/738
* Change default rocprof backend interface to rocprofv3
* Add MI 350 support in documentation
* Added known issue that MI 100 profiling will not work unless rocprofv1
is explicitly opted in
* Remove MI 50 soc gfx python class since MI 50 is not supported
[ROCm/rocprofiler-compute commit: d27ee69b52]
* Remove rocscope related code
* Add deprecation warning for database update mode which is used for grafana and mongodb functionality
[ROCm/rocprofiler-compute commit: cdd41dee40]
xuchen-amd
Fei Zheng
Pratik Basyal
vedithal-amd
cfallows-amd
fxmarty-amd
jamessiddeley-amd
David Galiffi
Kunal Malviya
dependabot[bot]
Jason Bonnell
Vladimir Indic
ywang103-amd