Skip to content

Conversation

@jgphpc
Copy link
Contributor

@jgphpc jgphpc commented Aug 29, 2019

Will fix UES-509

@vkarak
Copy link
Contributor

vkarak commented Sep 2, 2019

@jgphpc Can you put a more descriptive title to your PR?

@jgphpc jgphpc changed the title [test] report for advisor [test] Adding benchmarking mode to the Intel Advisor check Sep 2, 2019
@jgphpc jgphpc changed the title [test] Adding benchmarking mode to the Intel Advisor check [test] Add performance report to the Intel Advisor test Sep 2, 2019
Copy link
Contributor

@vkarak vkarak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@codecov-io
Copy link

codecov-io commented Sep 14, 2019

Codecov Report

Merging #916 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #916   +/-   ##
======================================
  Coverage    91.7%   91.7%           
======================================
  Files          79      79           
  Lines       10486   10486           
======================================
  Hits         9616    9616           
  Misses        870     870

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 224dc12...75e2461. Read the comment docs.

@vkarak vkarak merged commit 9aa14d4 into reframe-hpc:master Sep 14, 2019
@jgphpc jgphpc deleted the UES-509_advisor branch September 14, 2019 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants