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

[FEA] Qualification tool UI cosmetics #5517

Closed
amahussein opened this issue May 17, 2022 · 0 comments · Fixed by #5600
Closed

[FEA] Qualification tool UI cosmetics #5517

amahussein opened this issue May 17, 2022 · 0 comments · Fixed by #5600
Assignees
Labels
P0 Must have for release tools

Comments

@amahussein
Copy link
Collaborator

amahussein commented May 17, 2022

Following up on #5176 (with its PR #5470) , this issue lists all the remaining UI cosmetic requests.

Suggestions made by @mattahrens

  • RAPIDS Candidates (top box): put number right below the text to align with other top boxes; can leave % + text at bottom
  • GPU Opportunity (top box): add some designation related to task hours so it's clear that the units aren't wall clock durations; otherwise, it may be confusing that total in this box don't add up totals of the app metrics displayed in the dropdown
  • GPU Opportunity top box: can you sum up total durations and supported durations in wall clock (not task duration)? Unless I'm misinterpreting it

New issues raised from #5550:

  • found that the GPUSaveTime is not correctly rendered, and needs to be updated.
  • Cleanup the UI code: UI went through lots of experimental phases. Some code is not needed anymore. Also, review fieldnames
  • Add more details to the raw.html in order to be used as full detailed page.
@amahussein amahussein added feature request New feature or request ? - Needs Triage Need team to review and classify labels May 17, 2022
@amahussein amahussein added this to the May 2 - May 20 milestone May 17, 2022
@amahussein amahussein self-assigned this May 17, 2022
@sameerz sameerz added P0 Must have for release and removed ? - Needs Triage Need team to review and classify labels May 17, 2022
@mattahrens mattahrens added tools and removed P0 Must have for release labels May 17, 2022
@mattahrens mattahrens added the P0 Must have for release label May 24, 2022
@sameerz sameerz removed the feature request New feature or request label Jun 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 Must have for release tools
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants