ROCm / rocprofiler-compute

Advanced Profiling and Analytics for AMD Hardware
https://rocm.docs.amd.com/projects/omniperf/en/latest/
MIT License
135 stars 49 forks source link

Output data format #17

Closed ooreilly closed 1 year ago

ooreilly commented 1 year ago
omniperf -v
omniperf (1.0.3)

omniperf analyze -p workloads/kernel/mi200/ -o doesnotexist

--------
Analyze
--------

Saved Analysis folder exists

Why does omniperf analysis say: Saved Analysis folder exists ? The folder does not exist. If create the directory and rerun, I get:

IsADirectoryError: [Errno 21] Is a directory: 'doesnotexist'

I guess that omniperf says Saved analysis folder exists because I pointed it to the existing workload workloads/kernel/mi200.

The output that I get via -o is the stdout? It would be helpful it the data was in say sqlite, json, or even Python pickle file. Such an option would facilitate further analysis.

coleramos425 commented 1 year ago

The "Saved Analysis folder exists" is verbose output from Omniperf indicating that its successfully detected some results that we've cached in your workloads folder. For example image

This is more verbose output useful to the developer, but we can easily remove this if it makes things confusing. The -o option is expecting you to pass a name for the output file. For example

omniperf analyze -p workloads/kernel/mi200/ -o my_output.txt

In your situation the IsADirectoryError occurred because -o was expecting a filename or path with filename and you gave it a directory

coleramos425 commented 1 year ago

@ooreilly let me know if you have any follow-up questions or comments. Otherwise, I'll mark this issue as closed. Thanks!

ooreilly commented 1 year ago

@coleramos425 Thanks for the clarifications. I think it could be enabled when pass -V. I don't see have that information is useful in general. Can you perform an analysis without having a saved folder?

You can close this ticket, and I will open a feature request for saving the output data in different formats.

coleramos425 commented 1 year ago

Agreed. We'll move this into verbose output. Thanks for the suggestion!

And yes, you can perform analysis without having a saved analysis folder.