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

Show Aggregated Perf Results in Perf Graph View #106

Open
piyush286 opened this issue May 3, 2019 · 3 comments
Open

Show Aggregated Perf Results in Perf Graph View #106

piyush286 opened this issue May 3, 2019 · 3 comments
Labels
enhancement New feature or request

Comments

@piyush286
Copy link
Contributor

piyush286 commented May 3, 2019

Problem Description

With #73, we've added the ability to aggregate the perf results from multiple iterations. We should update Perf Graph View to use and display aggregated perf results in order to comply with the new design.

Proposed Changes

Besides the info that's already displayed, show all aggregated data for each build:

  • max
  • min
  • median
  • stddev
  • CI
  • iteration

Assigned Contributors

I'll be working with Sophia (@sophiaxu0424) from my team to add this feature.

@karianna karianna added this to To do in aqa-test-tools via automation May 6, 2019
@karianna karianna added the enhancement New feature or request label May 6, 2019
@llxia
Copy link
Contributor

llxia commented Jun 19, 2019

We should break this into several steps:

@sophiaxu0424
Copy link
Contributor

sophiaxu0424 commented Sep 24, 2019

  • make sure one Perf graph (i.e., ODM, SPECjbb, DT3, etc) works with the updated data structure

I have updated ODM.jsx and added extra info from aggregatiInfo set,
having a new graph looks as below:
Screen Shot 2019-11-14 at 4 38 59 PM

@piyush286 @llxia

jiayaolinn added a commit to jiayaolinn/openjdk-test-tools that referenced this issue Dec 16, 2019
- Based on ODM.jsx, added a perf graph for AcmeAir
- Added parsers for adding information to Node version

Issue adoptium#119
Issue adoptium#106

Signed-off-by: Jiayao Lin <jiayaolinn@gmail.com>
jiayaolinn added a commit to jiayaolinn/openjdk-test-tools that referenced this issue Dec 19, 2019
- Based on ODM.jsx, added a perf graph for AcmeAir
- Added parsers for adding information to Node version

Issue adoptium#119
Issue adoptium#106

Signed-off-by: Jiayao Lin <jiayaolinn@gmail.com>
jiayaolinn added a commit to jiayaolinn/openjdk-test-tools that referenced this issue Dec 24, 2019
- Based on ODM.jsx, added a perf graph for AcmeAir
- Added parsers for adding information to Node version

Issue adoptium#119
Issue adoptium#106

Signed-off-by: Jiayao Lin <jiayaolinn@gmail.com>
jiayaolinn added a commit to jiayaolinn/openjdk-test-tools that referenced this issue Jan 14, 2020
- Based on ODM.jsx, added a perf graph for AcmeAir
- Added parsers for adding information to Node version

Issue adoptium#119
Issue adoptium#106

Signed-off-by: Jiayao Lin <jiayaolinn@gmail.com>
aqa-test-tools automation moved this from To do to Done Feb 28, 2020
@piyush286
Copy link
Contributor Author

piyush286 commented Feb 29, 2020

Only ODM and AcmeAir graphs are updated to use aggregated perf results. Liberty and SPECjbb2015 graphs haven't been updated. Since we aren't really using any perf graphs for perf monitoring due to their current limitations, we can probably fix those after working on #118, not requiring any special changes for them.

@piyush286 piyush286 reopened this Feb 29, 2020
aqa-test-tools automation moved this from Done to In progress Feb 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
aqa-test-tools
  
In progress
Development

No branches or pull requests

4 participants