Skip to content

Exclude bottleneck tests in PyTorch#2450

Merged
akesandgren merged 1 commit intoeasybuilders:developfrom
Flamefire:pytorch_skip_bottleneck
Jun 1, 2021
Merged

Exclude bottleneck tests in PyTorch#2450
akesandgren merged 1 commit intoeasybuilders:developfrom
Flamefire:pytorch_skip_bottleneck

Conversation

@Flamefire
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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