Commit-Graf

517 Incheckningar

Upphovsman SHA1 Meddelande Datum
Karl W Schulz 093a4511ee update logic to detect roofline binaries in two alternate paths
depending on whether user is running within local clone or from form
install.

Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-04-21 14:53:14 -05:00
Karl W Schulz 3c562588ff update logic to detect VERSION file to accommodate rocm packaging;
check two locations to cover case where user is running within local
git clone directly or alternatively, from package install.

Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-04-21 14:53:14 -05:00
Karl W Schulz 65967658e9 fix execution error when OMNIPERF_COLOR env is set; update coloring to
support four modes:

(0) - no coloring and no loglevel delimiters
(1) - colored loglevel delimiters
(2) - non-colored loglevel delimiters
(3) - fully colored messages for all levels besides INFO

Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-04-04 14:44:51 -05:00
coleramos425 9403dce667 Define a README for /src subdir
Signed-off-by: coleramos425 <colramos@amd.com>
2024-04-01 14:30:21 -05:00
coleramos425 aac471c0fa Reorganizing docs runner and setting archive subdir for old docs
Signed-off-by: coleramos425 <colramos@amd.com>
2024-04-01 14:30:21 -05:00
coleramos425 9c93449cc7 Remove hardcoded URLs from docs in favor of relative links
Signed-off-by: coleramos425 <colramos@amd.com>
2024-04-01 14:30:21 -05:00
coleramos425 e0556f32ab Move dispatch id patch to proper util func in utils. Enable in rocprofv2 post-processing
Signed-off-by: coleramos425 <colramos@amd.com>
2024-04-01 14:30:21 -05:00
Nick Curtis a1017b68e9 implement rocprofv2 workaround for dispatch ids (#336)
* implement rocprofv2 workaround for dispatch ids

Signed-off-by: Nicholas Curtis <nicurtis@amd.com>

* formatting

Signed-off-by: Nicholas Curtis <nicurtis@amd.com>

---------

Signed-off-by: Nicholas Curtis <nicurtis@amd.com>
Co-authored-by: Nicholas Curtis <nicurtis@amd.com>
2024-04-01 14:30:21 -05:00
coleramos425 1a3bdad90a Adding documentation for global command line options
Signed-off-by: coleramos425 <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd 6cc8f0154f Restore OMNIPERF_COLOR global to disable default log coloring
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd b1d0b3905c Extending log coloring to message text. Enable by default.
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd 78c48eaed5 Remove superfluous logging statement
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd f47b8d2547 Throw more helpful warning on missing csv in analyze mode (#313)
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
coleramos425 706afa7f37 Register TRACE loglevel ahead of parse_args to avoid error in --specs printout
Signed-off-by: coleramos425 <colramos@amd.com>
2024-04-01 14:30:21 -05:00
Cole Ramos f7394bf272 Revert #333 deterministic ordering (#337)
* Revert "formatting"

This reverts commit e387561791898467371b6219921f10447d3486b3.

* Revert "Implement deterministic ordering of perfmon"

This reverts commit 2a24600a37c8002838145962a957cea447555b0c.
2024-04-01 14:30:21 -05:00
Nicholas Curtis f6f6165379 formatting
Signed-off-by: Nicholas Curtis <nicurtis@amd.com>
2024-04-01 14:30:21 -05:00
Nicholas Curtis 97af96e435 Implement deterministic ordering of perfmon
Signed-off-by: Nicholas Curtis <nicurtis@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd 3928f36d5e Adding redirect to help button in standalone GUI
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd 33e763178e Removing unused modules
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd 3d8b748924 Documentation enhancements and removing watermark
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd 73ac6c9561 Adding comments to roofline and changing output file name for specificity
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd cf36fb7fbf Implementing docs feedback with typos and minor enhancements
Co-authored-by: Jose Santos <josantos@amd.com>
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd 6cfdca93b9 Correct typo in Mi100 metric defs for missing SQ_INSTS_VMEM
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd 20670da2b7 Responding to first round of Ben R.'s docs feedback.\n\nThis round includes fixes for comments up to the 'Performance Model' section. I will need to work with our documentation group to respond to those higher level comments.
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd dfdd8dd21d Add timestamp override to rocprofv2 and format Python
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
colramos-amd 482fd6f2ca Create general_options() func to reduce code in parser
Signed-off-by: colramos-amd <colramos@amd.com>
2024-04-01 14:30:21 -05:00
Nick Curtis d72ef52208 Fix for #300 (#331)
On some versions of numpy/pandas, applying a str map to an empty series doesn't change the type.
As a result, when adding a float to a string, sometimes numpy dies with a uadd error.
We fix this by explicitly casting to a str first

Signed-off-by: Nicholas Curtis <nicurtis@amd.com>
Co-authored-by: Nicholas Curtis <nicurtis@amd.com>
2024-04-01 14:30:21 -05:00
David Galiffi ddb5c0ccc9 Updated links in documentation. (#329)
Updated to reflect new GitHub organization.
Fixed broken links to GitHub pages.

Signed-off-by: David Galiffi <David.Galiffi@amd.com>
2024-04-01 14:30:21 -05:00
David Galiffi f5712875aa Updated links in documentation. (#328)
Updated to reflect new GitHub organization.
Fixed broken links to GitHub pages.

Signed-off-by: David Galiffi <David.Galiffi@amd.com>
2024-03-21 10:14:37 -05:00
colramos-amd 62221383a7 Fixing visualization for Compute Pipeline SOL. Need to consider 'Pct of Peak' rather than 'Avg'
Signed-off-by: colramos-amd <colramos@amd.com>
2024-03-18 14:28:14 -05:00
Karl W Schulz 4fc30eb5c4 apply formatter
Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-03-15 16:40:13 -05:00
Karl W. Schulz 03ddba538d update doc watermark for release candidate
Signed-off-by: Karl W. Schulz <karl.schulz@amd.com>
2024-03-15 16:31:42 -05:00
Karl W Schulz f4070e3654 apply formatter
Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-03-15 14:04:49 -05:00
colramos-amd b436415784 Update docs for MI300 support
Signed-off-by: colramos-amd <colramos@amd.com>
2024-03-15 12:48:00 -05:00
colramos-amd 2dcad38028 Update roofline distro_map for Ubuntu22.04 (#319)
Signed-off-by: colramos-amd <colramos@amd.com>
2024-03-15 12:47:30 -05:00
Karl W Schulz 69de978d35 tweak logger formatter to suppor OMNIPERF_COLOR=2 which includes the
loglevel name in output, but without color

Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-03-15 09:31:22 -05:00
Karl W Schulz 357d225716 tighten up [analyze] help to be consistent with [profile] help
Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-03-15 09:31:22 -05:00
Karl W Schulz c29f28e25e bug fix for df tracing output - add more function demarcation
Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-03-15 09:31:22 -05:00
Karl W Schulz 76121f20a9 enable more function demarcation for analyze tracing
Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-03-15 09:31:22 -05:00
Karl W Schulz 3438bc9d0f logger update: assign color for TRACE level
Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-03-14 17:37:30 -05:00
Karl W Schulz 69af2ffc83 update logging formatter handling for consistent behavior with error messsages
Signed-off-by: Karl W Schulz <karl.schulz@amd.com>
2024-03-14 17:37:30 -05:00
Nick Curtis b610e1c27b fixup MI300 MFMA rate disagreement between SOL and pipeline (#315)
Signed-off-by: Nicholas Curtis <nicurtis@amd.com>
Co-authored-by: Nicholas Curtis <nicurtis@amd.com>
2024-03-14 10:41:35 -05:00
colramos-amd bfef420099 Update doc samples for new logging output
Signed-off-by: colramos-amd <colramos@amd.com>
2024-03-12 20:17:46 -05:00
colramos-amd 2acc8e98c1 Found a bug in kernel filtering and fixed
Signed-off-by: colramos-amd <colramos@amd.com>
2024-03-12 18:21:35 -05:00
colramos-amd cac4b1aff5 Python format
Signed-off-by: colramos-amd <colramos@amd.com>
2024-03-12 16:47:18 -05:00
colramos-amd fa7c82de75 Throw error if user analyzes two of same path (#260, #67)
Signed-off-by: colramos-amd <colramos@amd.com>
2024-03-12 16:46:36 -05:00
colramos-amd e7cc326d5a Fix missing argv in console_warning()
Signed-off-by: colramos-amd <colramos@amd.com>
2024-03-12 16:44:02 -05:00
JoseSantosAMD efb8f131ec change stubbed values to hbm bw (#302)
* change stubbed values to hbm bw

Signed-off-by: Jose Santos <josantos@amd.com>

* Reformat

Signed-off-by: Jose Santos <josantos@amd.com>

* Update analysis_webui.py

remove print statements

Signed-off-by: JoseSantosAMD <87447437+JoseSantosAMD@users.noreply.github.com>

* Update src/omniperf_analyze/analysis_webui.py

Co-authored-by: Cole Ramos <colramos@amd.com>
Signed-off-by: JoseSantosAMD <87447437+JoseSantosAMD@users.noreply.github.com>

* Update src/omniperf_analyze/analysis_webui.py

Co-authored-by: Cole Ramos <colramos@amd.com>
Signed-off-by: JoseSantosAMD <87447437+JoseSantosAMD@users.noreply.github.com>

---------

Signed-off-by: Jose Santos <josantos@amd.com>
Signed-off-by: JoseSantosAMD <87447437+JoseSantosAMD@users.noreply.github.com>
Co-authored-by: Cole Ramos <colramos@amd.com>
2024-03-12 16:08:30 -05:00
JoseSantosAMD 4424f320ad Kernel no overwrite (#303)
* Removing kernel name shortener from profile

-  Kernel name shortener now in analyze
-  db import uses pymongo to import

Signed-off-by: Jose Santos <josantos@amd.com>

* collections now added using pymongo

Signed-off-by: JoseSantosAMD <Jose.Santos@amd.com>

* kernel_name_shortener shortens dataframes and not a workload

-  argparser: --kernel-verbose to connection group
-  analysis_cli, profiler_rocprof_v2: remove kernel_name_shortener, so that it doesnt overwrite workload
-  db_import: demangle kernel_name dfs

Signed-off-by: JoseSantosAMD <Jose.Santos@amd.com>

* fix db name in pymongo command

Signed-off-by: JoseSantosAMD <Jose.Santos@amd.com>

* reformat

Signed-off-by: JoseSantosAMD <Jose.Santos@amd.com>

* fix typo

Signed-off-by: JoseSantosAMD <Jose.Santos@amd.com>

* remove checker for verbose in profiler_base

Signed-off-by: Jose Santos <josantos@amd.com>

* kernel_name_shortener now called in create_df_kerenel_top_stats and
create_df_pmc

Signed-off-by: Jose Santos <josantos@amd.com>

* add kernel_verbose to analysis_webui, remove stubbed values

Signed-off-by: JoseSantosAMD <Jose.Santos@amd.com>

* Update src/utils/kernel_name_shortener.py

keep console_debug

Co-authored-by: Cole Ramos <colramos@amd.com>
Signed-off-by: JoseSantosAMD <87447437+JoseSantosAMD@users.noreply.github.com>

* Update src/utils/kernel_name_shortener.py

Co-authored-by: Cole Ramos <colramos@amd.com>
Signed-off-by: JoseSantosAMD <87447437+JoseSantosAMD@users.noreply.github.com>

---------

Signed-off-by: Jose Santos <josantos@amd.com>
Signed-off-by: JoseSantosAMD <Jose.Santos@amd.com>
Signed-off-by: JoseSantosAMD <87447437+JoseSantosAMD@users.noreply.github.com>
Co-authored-by: Cole Ramos <colramos@amd.com>
2024-03-12 15:54:52 -05:00
colramos-amd 706d3bb9f0 Update DB module for new sysinfo keys
Signed-off-by: colramos-amd <colramos@amd.com>
2024-03-12 12:24:49 -05:00