Skip to content

Conversation

@hurricane642
Copy link
Contributor

This is a continuation of the work on the test library. This PR work has been done with PyTorch tests. A new file base_check.py has been added, introducing the main class for the PyTorch test, as well as the CSCS tests inherited from it.

@pep8speaks
Copy link

pep8speaks commented Sep 9, 2021

Hello @hurricane642, Thank you for updating!

Cheers! There are no PEP8 issues in this Pull Request!Do see the ReFrame Coding Style Guide

Comment last updated at 2021-11-30 20:47:21 UTC

@jenkins-cscs
Copy link
Collaborator

Can I test this patch?

@vkarak vkarak added this to the ReFrame Sprint 21.09.2 milestone Sep 28, 2021
@vkarak vkarak changed the title [test] Add generic PyTorch check for the cscs-supported apps into hpctestlib/apps [testlib] Add generic PyTorch check for the cscs-supported apps into hpctestlib/apps Nov 9, 2021
@codecov-commenter
Copy link

codecov-commenter commented Nov 30, 2021

Codecov Report

Merging #2183 (6cf6423) into master (bd478fe) will not change coverage.
The diff coverage is n/a.

❗ Current head 6cf6423 differs from pull request most recent head 3e01030. Consider uploading reports for the commit 3e01030 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2183   +/-   ##
=======================================
  Coverage   86.12%   86.12%           
=======================================
  Files          56       56           
  Lines       10208    10208           
=======================================
  Hits         8792     8792           
  Misses       1416     1416           

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 bd478fe...3e01030. Read the comment docs.

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.

Just a couple of minor comments still.

@vkarak
Copy link
Contributor

vkarak commented Nov 30, 2021

ok to test

@vkarak vkarak changed the title [testlib] Add generic PyTorch check for the cscs-supported apps into hpctestlib/apps [testlib] Add PyTorch library test and base CSCS test on top of it Nov 30, 2021
@vkarak vkarak merged commit 2b2b499 into reframe-hpc:master Nov 30, 2021
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.

6 participants