Skip to content

Conversation

@ChristopherBignamini
Copy link
Contributor

This should fix issue UES-734

@vkarak
Copy link
Contributor

vkarak commented Feb 19, 2020

@ChristopherBignamini Why is this a wip?

@vkarak vkarak changed the title [dom][wip][test] GridTools Test failures on Dom [wip][test] GridTools Test failures on Dom Feb 19, 2020
@vkarak vkarak self-requested a review February 19, 2020 11:00
@vkarak vkarak assigned vkarak and ChristopherBignamini and unassigned vkarak Feb 19, 2020
@vkarak vkarak added this to the ReFrame sprint 20.02 milestone Feb 19, 2020
@ChristopherBignamini
Copy link
Contributor Author

@vkarak I can remove the wip, bad use of the tag.

@vkarak vkarak changed the title [wip][test] GridTools Test failures on Dom [test] GridTools Test failures on Dom Feb 20, 2020
@vkarak vkarak changed the title [test] GridTools Test failures on Dom [test] Fix GridTools test failures on Dom Feb 20, 2020
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

@vkarak
Copy link
Contributor

vkarak commented Feb 20, 2020

@jenkins-cscs retry none

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1181   +/-   ##
=======================================
  Coverage   92.02%   92.02%           
=======================================
  Files          81       81           
  Lines       11739    11739           
=======================================
  Hits        10803    10803           
  Misses        936      936

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 9e20298...9dd3c31. Read the comment docs.

@vkarak vkarak merged commit bd6b6a3 into reframe-hpc:master Feb 20, 2020
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.

3 participants