Skip to content

fix version and source for Seurat, add extensions required by Seurat in R 3.4.0 easyconfig#4889

Merged
vanzod merged 2 commits intoeasybuilders:developfrom
boegel:fix_seurat
Aug 9, 2017
Merged

fix version and source for Seurat, add extensions required by Seurat in R 3.4.0 easyconfig#4889
vanzod merged 2 commits intoeasybuilders:developfrom
boegel:fix_seurat

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Jul 13, 2017

Loading Seurat in R results in the following error:

> library(Seurat)
Error: package ‘Seurat’ was built for x86_64-apple-darwin13.4.0

This went undetected because of a bug in the EasyBuild framework where the library(...) import checks for R extensions were basically being skipped, see easybuilders/easybuild-framework#2276.

The problem was the we were using the binary release of Seurat, rather than the source tarball...

As it turns out, several R extensions were also missing as dependencies.

@boegel
fix version and source for Seurat (use sources, not binary release), …
41d89ea 
…add extensions required by Seurat in R 3.4.0 easyconfig
@boegel boegel added this to the 3.4.0 milestone Jul 13, 2017
@boegel boegel mentioned this pull request Jul 13, 2017
Merged
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jul 13, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2541.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/a611caa8da1570cab125e718c7ffe8f0 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jul 13, 2017

Tested on top of easybuilders/easybuild-framework#2276.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jul 13, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2035.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/9ff590bac89bcd5680f08c005486424c for a full test report.

vanzod
vanzod requested changes Jul 13, 2017
View reviewed changes
Copy link
Copy Markdown
Member

@vanzod vanzod left a comment

Choose a reason for hiding this comment

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

Add checksums

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 6, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2011.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/cf3a6d7e8a59fe287c8be0f0d18bb746 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 6, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2510.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/7f3f8f95f91cce149d6edd18bcd6ffc3 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 6, 2017

@vanzod Good to go now?

@vanzod
Copy link
Copy Markdown
Member

vanzod commented Aug 9, 2017

Going in, thanks @boegel!

@vanzod vanzod merged commit 810c783 into easybuilders:develop Aug 9, 2017
@boegel boegel deleted the fix_seurat branch August 9, 2017 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vanzod vanzod vanzod approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.4.0

Development

Successfully merging this pull request may close these issues.

2 participants

@boegel @vanzod