Skip to content

add toolchain nvompic #3735

New issue

Have a question about this project? Sign up for a free account to open an issue and contact its maintainers and the community.

By clicking “Sign up for ”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on ? Sign in to your account

Merged
merged 8 commits into from
Jun 28, 2021
Merged

Conversation

SebastianAchilles
Copy link
Member

@SebastianAchilles SebastianAchilles commented Jun 10, 2021

This adds the nvompi toolchain: NVHPC + OpenMPI

boegel
boegel previously requested changes Jun 11, 2021
@boegel
Copy link
Member

@SebastianAchilles I'm not sure what's going on with the tests here...

@boegelboegel added this to the 4.x milestone Jun 11, 2021
@SebastianAchillesSebastianAchilles changed the title add toolchain nompi add toolchain nvompic Jun 11, 2021
@boegelbot
Copy link

@SebastianAchilles: Tests failed in Actions, see https://.com/easybuilders/easybuild-framework/actions/runs/978318613
Output from first failing test suite run:

FAIL: test_docstrings (test.framework.general.GeneralTest)
Make sure tags included in docstrings are correctly formatted.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/runner/1cbbcd4839df37c2ac90961b94f8b7b42f5039f5/lib/python2.7/site-packages/test/framework/general.py", line 124, in test_docstrings
    self.assertFalse(regex.search(txt), "No match for '%s' in %s" % (regex.pattern, path))
AssertionError: <_sre.SRE_Match object at 0x7f29c5f87d50> is not false : No match for '@author' in /tmp/runner/1cbbcd4839df37c2ac90961b94f8b7b42f5039f5/lib/python2.7/site-packages/easybuild/toolchains/nvompic.py

----------------------------------------------------------------------
Ran 782 tests in 979.604s

FAILED (failures=1)
ERROR: Not all tests were successful.

bleep, bloop, I'm just a bot (boegelbot v20200716.01)
Please talk to my owner @boegel if you notice you me acting stupid),
or submit a pull request to https://.com/boegel/boegelbot fix the problem.

@easybuilderseasybuilders deleted a comment from boegelbot Jun 28, 2021
@easybuilderseasybuilders deleted a comment from boegelbot Jun 28, 2021
@easybuilderseasybuilders deleted a comment from boegelbot Jun 28, 2021
@akesandgrenakesandgren dismissed boegel’s stale review June 28, 2021 10:59

Request for changes implemented

@easybuilderseasybuilders deleted a comment from boegelbot Jun 28, 2021
@akesandgren
Copy link
Contributor

Going in, thanks @SebastianAchilles!

Sign up for free to join this conversation on . Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants