Skip to content

{math}[GCCcore/6.4.0] Eigen v3.3.4#4928

Merged
verdurin merged 2 commits intoeasybuilders:developfrom
JackPerdue:20170721083605_new_pr_Eigen334
Jul 30, 2017
Merged

{math}[GCCcore/6.4.0] Eigen v3.3.4#4928
verdurin merged 2 commits intoeasybuilders:developfrom
JackPerdue:20170721083605_new_pr_Eigen334

Conversation

@JackPerdue
Copy link
Copy Markdown
Contributor

(created using eb --new-pr)

@verdurin verdurin added this to the 3.4.0 milestone Jul 22, 2017
@verdurin
Copy link
Copy Markdown
Member

Test report by @verdurin
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
ca004.camp.thecrick.org - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2640 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/039cf21fdd32523b5971bf48dd325417 for a full test report.

Copy link
Copy Markdown
Member

@verdurin verdurin left a comment

Choose a reason for hiding this comment

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

Otherwise, looks good

'IterativeLinearSolvers', 'Jacobi', 'OrderingMethods',
'PaStiXSupport', 'PardisoSupport', 'SparseCholesky',
'SparseCore', 'StdDeque', 'StdList',
'SuperLUSupport', 'UmfPackSupport'])
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.

Forgive my ignorance, but don't these repeat what's in the Eigen easyblock?

@JackPerdue
Copy link
Copy Markdown
Contributor Author

I honestly don't remember why I did that Sanity checks removed.

Copy link
Copy Markdown
Member

@verdurin verdurin left a comment

Choose a reason for hiding this comment

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

Looks good

@verdurin
Copy link
Copy Markdown
Member

Merging, thanks @JackPerdue

@verdurin verdurin merged commit 7c2e069 into easybuilders:develop Jul 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants