Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

{bio,math}[foss/2020b] R-bundle-Bioconductor v3.12, ncdf4 v1.17 #12151

Conversation

branfosj
Copy link
Member

@branfosj branfosj commented Feb 12, 2021

(created using eb --new-pr)

  • Remove the MariaDB-connector-c dependency as there is no RMySQL in this EC and it has not been in the Bioconductor ECs for quite a few versions.
  • Remove sigaR and MotIV - both are listed as deprecated in Bioconductor and attempting to download the latest version fails. (See https://bioconductor.org/packages/release/bioc/html/sigaR.html and http://www.bioconductor.org/packages/release/bioc/html/MotIV.html)
  • Add signalToolbox (we've had a request for this to be added). Everything from agricolae to the end is a dependency for this.
  • Add BH, with a newer version than in R-4.0.3-foss-2020b.eb, as cytolib requires the newer version.
  • Add sctransform, with a newer version than in R-4.0.3-foss-2020b.eb, as Seurat requires the newer version.

@branfosj branfosj marked this pull request as draft February 12, 2021 16:16
@boegel boegel added this to the release after 4.3.3 milestone Feb 12, 2021
@boegel
Copy link
Member

boegel commented Feb 12, 2021

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

@boegel: Request for testing this PR well received on generoso

PR test command 'EB_PR=12151 EB_ARGS= /apps/slurm/default/bin/sbatch --job-name test_PR_12151 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 13809

Test results coming soon (I hope)...

- notification for comment with ID 778480400 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@branfosj branfosj marked this pull request as ready for review February 12, 2021 23:24
@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
generoso-c1-s-1 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/bc7462a355fe5252f27246ccd54aee18 for a full test report.

@branfosj
Copy link
Member Author

Test report by @branfosj
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
bear-pg0211u03a.bear.cluster - Linux centos linux 7.8.2003, x86_64, Intel(R) Xeon(R) Gold 6248 CPU @ 2.50GHz (cascadelake), Python 3.6.8
See https://gist.github.com/7aba4e6cecf97d539f89ee4f5e468eb2 for a full test report.

@branfosj
Copy link
Member Author

Test report by @branfosj
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
bear-pg0305u05a.bear.cluster - Linux RHEL 8.2, POWER, 8335-GTX (power9le), Python 3.6.8
See https://gist.github.com/d9a2dd607fad71868dcc66f6c702d063 for a full test report.

Copy link
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Member

boegel commented Feb 13, 2021

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node3501.doduo.os - Linux RHEL 8.2, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/0f0377d2aea048e77cb572f9a718ecbb for a full test report.

@boegel
Copy link
Member

boegel commented Feb 13, 2021

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node2604.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/b67052c283f4e8a7bd86774e2d3bbdcb for a full test report.

@boegel
Copy link
Member

boegel commented Feb 13, 2021

Going in, thanks @branfosj!

@boegel boegel merged commit faa48f1 into easybuilders:develop Feb 13, 2021
@branfosj branfosj deleted the 20210212160459_new_pr_R-bundle-Bioconductor312 branch February 13, 2021 16:10
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.

3 participants