Skip to content

{lang} [foss/2016b] MXNet/0.9.3 (REVIEW)#4765

Merged
boegel merged 3 commits intoeasybuilders:developfrom
wpoely86:mxnet-foss
Jun 22, 2017
Merged

{lang} [foss/2016b] MXNet/0.9.3 (REVIEW)#4765
boegel merged 3 commits intoeasybuilders:developfrom
wpoely86:mxnet-foss

Conversation

@wpoely86
Copy link
Copy Markdown
Member

@wpoely86 wpoely86 commented Jun 22, 2017

Split of MXNet/0.9.3 with foss/2016b from #4346

The foss one works, the intel one currently doesn't.

Needs easybuilders/easybuild-easyblocks#1135

@wpoely86
Split of MXNet/0.9.3 with foss/2016b from easybuilders#4346
ba0c1a4 
The foss one works, the intel one currently doesn't.
@wpoely86
Copy link
Copy Markdown
Member Author

wpoely86 commented Jun 22, 2017

Test report by @wpoely86
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
nic81 - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2680 v4 @ 2.40GHz, Python 2.7.5
See https://gist.github.com/40287d93ee6481acf8180434040d73ec for a full test report.

@boegel boegel added this to the 3.3.0 milestone Jun 22, 2017
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 22, 2017

@wpoely86 Please fix these minor style issues:

$ eb --from-pr 4765 --check-style
== temporary log file in case of crash /tmp/eb-sdR_GV/easybuild-71oh1c.log

*** WARNING: Using easyconfigs from unstable (pending/failed tests or merge conflict) PR #4765 ***

Running style check on 1 easyconfig(s)...
/tmp/eb-sdR_GV/files_pr4765/m/MXNet/MXNet-0.9.3-foss-2016b-Python-2.7.12.eb:16:27: E261 at least two spaces before inline comment
/tmp/eb-sdR_GV/files_pr4765/m/MXNet/MXNet-0.9.3-foss-2016b-Python-2.7.12.eb:20:71: E261 at least two spaces before inline comment
[FAIL] /tmp/eb-sdR_GV/files_pr4765/m/MXNet/MXNet-0.9.3-foss-2016b-Python-2.7.12.eb
ERROR: One or more style checks FAILED!

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Jun 22, 2017

Travis test report: 8/8 runs failed - see https://travis-ci.org/hpcugent/easybuild-easyconfigs/builds/245728046

Only showing partial log for 1st failed test suite run 5068.1;
full log at https://travis-ci.org/hpcugent/easybuild-easyconfigs/jobs/245728047

...
ERROR: Test for parsing of easyconfig MXNet-0.9.3-foss-2016b-Python-2.7.12.eb
----------------------------------------------------------------------
Traceback (most recent call last):
  File "<string>", line 1, in innertest
  File "/home/travis/build/hpcugent/easybuild-easyconfigs/test/easyconfigs/easyconfigs.py", line 201, in template_easyconfig_test
    ecs = process_easyconfig(spec)
  File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/easybuild_framework-3.3.0.dev0-py2.6.egg/easybuild/framework/easyconfig/easyconfig.py", line 1263, in process_easyconfig
    raise EasyBuildError("Failed to process easyconfig %s: %s", spec, err.msg)
EasyBuildError: "Failed to process easyconfig /home/travis/build/hpcugent/easybuild-easyconfigs/easybuild/easyconfigs/m/MXNet/MXNet-0.9.3-foss-2016b-Python-2.7.12.eb: No software-specific easyblock 'EB_MXNet' found for MXNet"

----------------------------------------------------------------------
Ran 7253 tests in 149.856s

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 22, 2017

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2050.delcatty.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/5b5f3fb17a1326b8ac2e2c93b2fc29cd for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 22, 2017

lgtm

@boegel boegel mentioned this pull request Jun 22, 2017
Merged
1 task
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 22, 2017

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2441.golett.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/226f25ea5e20055204efae2851cc4218 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 22, 2017

@wpoely86 Please fix filename (or versionsuffix):

AssertionError: Filename '/home/travis/build/hpcugent/easybuild-easyconfigs/easybuild/easyconfigs/m/MXNet/MXNet-0.9.3-foss-2016b-Python-2.7.12.eb' of parsed easyconfig matches expected filename 'MXNet-0.9.3-foss-2016b-Python-2.7.12-R-3.3.3.eb'

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 22, 2017

Going in, thanks @wpoely86!

@boegel boegel merged commit 68fe20b into easybuilders:develop Jun 22, 2017
@wpoely86 wpoely86 deleted the mxnet-foss branch June 22, 2017 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.3.0

Development

Successfully merging this pull request may close these issues.

3 participants

@wpoely86 @boegel @boegelbot