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

Causal profiling GUI: support startup w/o workload #249

Merged
merged 3 commits into from
Feb 16, 2023

Conversation

jrmadsen
Copy link
Collaborator

No description provided.

- some renaming of functions
- if empty data, empty div is returned
- fix errors from applying sort/filters when data is empty
@jrmadsen
Copy link
Collaborator Author

  • Support for directories is incorrect. Currently it just grabs the first entry in a glob instead of reading all of the files

@jrmadsen jrmadsen merged commit 81d182e into ROCm:develop Feb 16, 2023
@jrmadsen jrmadsen deleted the gui-workload-startup branch February 16, 2023 21:15
jrmadsen added a commit that referenced this pull request Mar 21, 2023
* Gui can be started with no workload.

- some renaming of functions
- if empty data, empty div is returned

* workloads can be directory also

* filtering errors fix

- fix errors from applying sort/filters when data is empty

---------

Co-authored-by: JoseSantosAMD <Jose.Santos@amd.com>
jrmadsen added a commit that referenced this pull request Mar 22, 2023
* Gui can be started with no workload.

- some renaming of functions
- if empty data, empty div is returned

* workloads can be directory also

* filtering errors fix

- fix errors from applying sort/filters when data is empty

---------

Co-authored-by: JoseSantosAMD <Jose.Santos@amd.com>
jrmadsen added a commit to JoseSantosAMD/omnitrace that referenced this pull request Apr 3, 2023
* Gui can be started with no workload.

- some renaming of functions
- if empty data, empty div is returned

* workloads can be directory also

* filtering errors fix

- fix errors from applying sort/filters when data is empty

---------

Co-authored-by: JoseSantosAMD <Jose.Santos@amd.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants