Skip to content

Conversation

@jgphpc
Copy link
Contributor

@jgphpc jgphpc commented Jun 14, 2018

Fixes #328.

Signed-off-by: jgp <jgp@cscs.ch>
@jgphpc jgphpc self-assigned this Jun 14, 2018
@jgphpc jgphpc requested a review from vkarak June 14, 2018 09:44
@codecov-io
Copy link

Codecov Report

Merging #330 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #330      +/-   ##
==========================================
+ Coverage   91.18%   91.21%   +0.02%     
==========================================
  Files          67       67              
  Lines        7941     7941              
==========================================
+ Hits         7241     7243       +2     
+ Misses        700      698       -2
Impacted Files Coverage Δ
reframe/core/config.py 84.54% <0%> (+1.81%) ⬆️

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 fd18a5e...c008093. Read the comment docs.

@vkarak vkarak changed the title ddt_check_Cuda fails after maintenance Fix CUDA DDT check after upgrade to CUDA 9 Jun 14, 2018
@vkarak vkarak added this to the ReFrame sprint 2018w23 milestone Jun 14, 2018
@vkarak vkarak requested a review from teojgo June 14, 2018 11:28
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. Thanks @jgphpc!

}
self.maintainers = ['MK', 'JG']
self.tags = {'production'}
self.post_run = ['ddt -V ; which ddt ;']
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this line needed just for checking or can we put another sanity check for this?

@jgphpc
Copy link
Contributor Author

jgphpc commented Jun 14, 2018 via email

@vkarak vkarak merged commit 1d3b32b into master Jun 14, 2018
@vkarak vkarak deleted the ddt+cuda9 branch June 14, 2018 13:01
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.

5 participants