{data} [intel/2017b] bat 0.3.3, arrow 0.7.1 (REVIEW) by wpoely86 · Pull Request #5416 · easybuilders/easybuild-easyconfigs

wpoely86 · 2017-11-27T14:08:47Z

Needs:

{devel} [intel/2017b] Spark 2.2.0

wpoely86 · 2017-11-30T08:46:45Z

Test report by @wpoely86
FAILED
Build succeeded for 2 out of 4 (4 easyconfigs in this PR)
nic200 - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E7-8891 v4 @ 2.80GHz, Python 2.7.5
See https://gist.github.com/410459f1466ec74c33895fb423e0f994 for a full test report.

wpoely86 · 2017-11-30T09:20:46Z

Test report by @wpoely86
SUCCESS
Build succeeded for 4 out of 4 (4 easyconfigs in this PR)
nic200 - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E7-8891 v4 @ 2.80GHz, Python 2.7.5
See https://gist.github.com/dd95de78c52b278da36d21ba3614e31e for a full test report.

boegel · 2017-11-30T09:30:46Z

+toolchain = {'name': 'intel', 'version': '2017b'}
+
+sources = ['apache-arrow-%(version)s.tar.gz']
+source_urls = ['https://github.com/apache/arrow/archive/']


@wpoely86 please include a SHA256 checksum

boegel · 2017-11-30T09:31:06Z

+start_dir = 'cpp'
+separate_build_dir = True
+
+builddependencies = [('CMake', '3.9.5')]


please move up, right below sources block

boegel · 2017-11-30T09:31:32Z

+
+modextrapaths = {'PYTHONPATH': ['lib/python%(pyshortver)s/site-packages']}
+
+dependencies = [


please move up, same block as builddependencies

boegel · 2017-11-30T09:32:13Z

+toolchain = {'name': 'intel', 'version': '2017b'}
+
+# this is a bundle of Python packages
+exts_defaultclass = 'PythonPackage'


please move down, right above exts_list (and drop the comment, it's clear without it imho)

boegel · 2017-11-30T09:32:24Z

+    ('PyYAML', '3.12', versionsuffix),
+    ('scikit-learn', '0.19.1', versionsuffix),
+    ('arrow', '0.7.1', versionsuffix),
+    ('Spark', '2.2.0', '-Hadoop-2.6-Java-1.8.0_152%s' % versionsuffix),


please use %(versionsuffix)s

boegel · 2017-11-30T09:32:57Z

+    ('pathtools', '0.1.2', {
+        'source_urls': ['https://pypi.python.org/packages/source/p/pathtools'],
+        'checksums': [
+            '7c35c5421a39bb82e58018febd90e3b6e5db34c5443aaaf742b3f33d4655f1c0',  # pathtools-0.1.2.tar.gz


let's collapse this to a single line (--inject-checksums in develop should do so automagically)

boegel · 2017-11-30T09:33:12Z

+toolchain = {'name': 'GCCcore', 'version': '6.4.0'}
+
+source_urls = ['https://github.com/jemalloc/jemalloc/archive']
+sources = ['%(version)s.tar.gz']


please include SHA256 checksum

wpoely86 · 2017-11-30T10:13:20Z

Test report by @wpoely86
SUCCESS
Build succeeded for 4 out of 4 (4 easyconfigs in this PR)
nic200 - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E7-8891 v4 @ 2.80GHz, Python 2.7.5
See https://gist.github.com/638064c164faee0dd1102f5facbc9f9b for a full test report.

boegel · 2017-11-30T10:18:35Z

+
+modextrapaths = {'PYTHONPATH': ['lib/python%(pyshortver)s/site-packages']}
+
+dependencies = [


@wpoely86 Please move this up too, missed that one earlier...

boegel · 2017-11-30T10:23:27Z

Test report by @boegel
FAILED
Build succeeded for 2 out of 4 (4 easyconfigs in this PR)
node2454.golett.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/58473fddefc005c0da64c6790ee8fd6e for a full test report.

boegel · 2017-11-30T10:25:44Z

Test report by @boegel
FAILED
Build succeeded for 2 out of 4 (4 easyconfigs in this PR)
node2066.delcatty.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/938a2a54e9c719b362960e4b53673344 for a full test report.

wpoely86 · 2017-11-30T10:32:06Z

@boegel you must use develop from today

boegel · 2017-11-30T10:50:10Z

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

boegel · 2017-11-30T10:53:58Z

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

boegel · 2017-11-30T10:59:16Z

Going in, thanks @wpoely86!

wpoely86 added 2 commits November 27, 2017 15:07

Add arrow/0.7.1

3860faa

Add bat/0.3.3

92d4582

wpoely86 mentioned this pull request Nov 28, 2017

Allow use of start_dir in extensions easybuilders/easybuild-framework#2353

Merged

wpoely86 and others added 8 commits November 28, 2017 10:50

add jemalloc for GCCcore/6.4.0

315940b

Added scikit-learn/0.19.1 for intel/2017b with Python/3.6.3

cf4616a

Use correct jemalloc

fa55ece

install pyspark extension

29887f8

Merge pull request #37 from smoors/spark

3bfb834

{devel} [intel/2017b] Spark 2.2.0

Add pyspark to bat

99a8983

Fix typo and correct files in extension

f028f41

%(namelower)s in extension referers to extension name

b89ed87

Add binutils to jemalloc

3de878b

wpoely86 changed the title ~~{data} [intel/2017b] bat 0.3.3 (WIP)~~ {data} [intel/2017b] bat 0.3.3 (REVIEW) Nov 30, 2017

easybuilders deleted a comment from boegelbot Nov 30, 2017

boegel requested changes Nov 30, 2017

View reviewed changes

boegel added this to the 3.5.0 milestone Nov 30, 2017

boegel added new update labels Nov 30, 2017

Fix remarks

72320d1

boegel requested changes Nov 30, 2017

View reviewed changes

More remarks

039500f

boegel approved these changes Nov 30, 2017

View reviewed changes

boegel merged commit 2b3fcd0 into easybuilders:develop Nov 30, 2017

wpoely86 deleted the bat branch November 30, 2017 11:01

boegel changed the title ~~{data} [intel/2017b] bat 0.3.3 (REVIEW)~~ {data} [intel/2017b] bat 0.3.3, arrow 0.7.1 (REVIEW) Dec 13, 2017


		modextrapaths = {'PYTHONPATH': ['lib/python%(pyshortver)s/site-packages']}

		dependencies = [

Conversation

wpoely86 commented Nov 27, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

wpoely86 commented Nov 30, 2017

Uh oh!

wpoely86 commented Nov 30, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

wpoely86 commented Nov 30, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

boegel commented Nov 30, 2017

Uh oh!

boegel commented Nov 30, 2017

Uh oh!

wpoely86 commented Nov 30, 2017

Uh oh!

boegel commented Nov 30, 2017

Uh oh!

boegel commented Nov 30, 2017

Uh oh!

boegel commented Nov 30, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

wpoely86 commented Nov 27, 2017 •

edited

Loading