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

Exclude bottleneck tests in PyTorch #2450

Merged

Conversation

Flamefire
Copy link
Contributor

This excludes a test which runs some kind of profiler. That exits often with an error code but no output at all, so the reason for the failure is unknown. This seems to be common enough there exists an official env var to skip those tests

Copy link
Contributor

@akesandgren akesandgren left a comment

Choose a reason for hiding this comment

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

LGTM

@akesandgren
Copy link
Contributor

Going in, thanks @Flamefire!

@akesandgren akesandgren merged commit 835a5c6 into easybuilders:develop Jun 1, 2021
@Flamefire Flamefire deleted the pytorch_skip_bottleneck branch June 1, 2021 09:11
@boegel
Copy link
Member

boegel commented Jun 1, 2021

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS PyTorch-1.6.0-foss-2019b-Python-3.7.4.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3127.skitty.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/263e7e03f5bac11e7edac1e6faaffe8e for a full test report.

@boegel
Copy link
Member

boegel commented Jun 1, 2021

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS PyTorch-1.7.1-fosscuda-2020b.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3303.joltik.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6242 CPU @ 2.80GHz (cascadelake), Python 3.6.8
See https://gist.github.com/c2046b6c00e415b31129107b9fc81c13 for a full test report.

boegel added a commit to migueldiascosta/easybuild-easyblocks that referenced this pull request Jun 1, 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.

3 participants