Skip to content

{lib}[fosscuda/2018b] TensorRT v4.0.1.6#7584

Merged
boegel merged 4 commits intoeasybuilders:developfrom
akesandgren:20190212145812_new_pr_TensorRT4016
Feb 14, 2019
Merged

{lib}[fosscuda/2018b] TensorRT v4.0.1.6#7584
boegel merged 4 commits intoeasybuilders:developfrom
akesandgren:20190212145812_new_pr_TensorRT4016

Conversation

@akesandgren
Copy link
Copy Markdown
Contributor

@akesandgren akesandgren commented Feb 12, 2019

(created using eb --new-pr)
This easyconfig requires easybuilders/easybuild-easyblocks#1627

Comment thread easybuild/easyconfigs/t/TensorRT/TensorRT-4.0.1.6-fosscuda-2018b-Python-2.7.15.eb Outdated
Comment thread easybuild/easyconfigs/t/TensorRT/TensorRT-4.0.1.6-fosscuda-2018b-Python-2.7.15.eb Outdated
'source_tmpl': 'protobuf-python-%(version)s.tar.gz',
'source_urls': ['https://github.com/google/protobuf/releases/download/v%(version)s/'],
'start_dir': 'python',
'use_pip': False,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@akesandgren Why not use pip here, any specific reason?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copied from the tensorflow easyconfig, so just following your lead :-)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Checked it again, there's apparently a good reason for it, it fails with an error when you try to use pip:

Can't find required file: ../src/google/protobuf/descriptor.proto

@akesandgren
Copy link
Copy Markdown
Contributor Author

akesandgren commented Feb 13, 2019

Test report by @akesandgren
SUCCESS
Build succeeded for 2 out of 2 (1 easyconfigs in this PR)
b-an03.hpc2n.umu.se - Linux ubuntu 16.04, Intel(R) Xeon(R) CPU E5-2690 v4 @ 2.60GHz, Python 2.7.12
See https://gist.github.com/30c82c2bef49f490dc537c49452f3cac for a full test report.

Using easybuilders/easybuild-easyblocks#1627 to build

@easybuilders easybuilders deleted a comment from boegelbot Feb 14, 2019
@boegel boegel added this to the next release (3.8.2) milestone Feb 14, 2019
minor style fixes in TensorRT easyconfig + add comment to motivate not using 'pip' for protobuf-python
@akesandgren
Copy link
Copy Markdown
Contributor Author

Kicking travis

@akesandgren
Copy link
Copy Markdown
Contributor Author

Travis kicked

@akesandgren akesandgren reopened this Feb 14, 2019
@easybuilders easybuilders deleted a comment from boegelbot Feb 14, 2019
@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 14, 2019

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node3103.skitty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/2417b4bc74900b8f8d8dbd99584e7f23 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 14, 2019

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 14, 2019

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 14, 2019

Going in, thanks @akesandgren!

@boegel boegel merged commit fef8cc0 into easybuilders:develop Feb 14, 2019
@akesandgren akesandgren deleted the 20190212145812_new_pr_TensorRT4016 branch February 14, 2019 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants