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}[foss/2020b] ABRA2 v2.23, CRISPR-DAV v2.3.4, Excel-Writer-XLSX v1.09, FLASH v2.2.00, PRINSEQ v0.20.4, pysamstats v1.1.2 w/ Python 3.8.6 #13139

Merged

Conversation

deniskristak
Copy link
Contributor

@deniskristak deniskristak commented Jun 14, 2021

(created using eb --new-pr)

requires easybuilders/easybuild-easyblocks#2487 (CRISPR-DAV easyblock) + #13762 (naturalsort extension for R)

@deniskristak
Copy link
Contributor Author

please dont merge yet, ive added crispr-dav, but it still needs work

@easybuilders easybuilders deleted a comment from boegelbot Jun 15, 2021
@deniskristak
Copy link
Contributor Author

must be used with easybuilders/easybuild-easyblocks#2487

@deniskristak
Copy link
Contributor Author

pushed to easyblock PR as well

@deniskristak
Copy link
Contributor Author

ready for review, @boegel

@easybuilders easybuilders deleted a comment from boegelbot Aug 18, 2021
}

modextravars = {
"LC_ALL": "en_US.UTF-8",
Copy link
Member

Choose a reason for hiding this comment

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

@deniskristak Hmm, why is this needed? That looks a bit fishy, it doesn't belong in a module generated by EasyBuild...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ABRA2 needs it because of this:
mozack/abra2#25

Copy link
Member

Choose a reason for hiding this comment

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

OK, please add a comment with a reference to that issue then:

# required to work around localization bug, see https://github.com/mozack/abra2/issues/25
modextravars = {'LC_ALL': 'en_US.UTF-8'}

easybuild/easyconfigs/f/FLASH/FLASH-2.2.00-foss-2020b.eb Outdated Show resolved Hide resolved
better postinstallcmds
adding sanity_check_commands
removing, is an R extension
adding sanity_check_commands
removed naturalsort, added comment about it
@deniskristak
Copy link
Contributor Author

@boegel ready for review

@boegel boegel changed the title {tools}[foss/2020b] naturalsort v0.1.3, pysamstats v1.1.2, Excel-Writer-XLSX v1.09 w/ Python 3.8.6 {tools}[foss/2020b] ABRA2 v2.23, CRISPR-DAV v2.3.4, Excel-Writer-XLSX v1.09, FLASH v2.2.00, PRINSEQ v0.20.4, pysamstats v1.1.2 w/ Python 3.8.6 Aug 19, 2021
}

modextravars = {
"LC_ALL": "en_US.UTF-8",
Copy link
Member

Choose a reason for hiding this comment

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

OK, please add a comment with a reference to that issue then:

# required to work around localization bug, see https://github.com/mozack/abra2/issues/25
modextravars = {'LC_ALL': 'en_US.UTF-8'}

@boegel boegel changed the title {tools}[foss/2020b] ABRA2 v2.23, CRISPR-DAV v2.3.4, Excel-Writer-XLSX v1.09, FLASH v2.2.00, PRINSEQ v0.20.4, pysamstats v1.1.2 w/ Python 3.8.6 {bio}[foss/2020b] ABRA2 v2.23, CRISPR-DAV v2.3.4, Excel-Writer-XLSX v1.09, FLASH v2.2.00, PRINSEQ v0.20.4, pysamstats v1.1.2 w/ Python 3.8.6 Aug 19, 2021
@boegel boegel added this to the next release (4.4.2?) milestone Aug 19, 2021
@boegel boegel force-pushed the 20210614131815_new_pr_naturalsort013 branch from bd32a16 to 47df656 Compare August 19, 2021 15:00
@boegel
Copy link
Member

boegel commented Aug 19, 2021

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2487
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in total)
node3585.doduo.os - Linux RHEL 8.2, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/511e361bb35aef80ba46503b62939e36 for a full test report.

@boegel
Copy link
Member

boegel commented Aug 19, 2021

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2487
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in total)
node3111.skitty.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/90e614f57b3cca5a4eeb32b9439adba3 for a full test report.

@boegel
Copy link
Member

boegel commented Aug 19, 2021

@boegelbot please test @ generoso
EB_ARGS="--include-easyblocks-from-pr 2487"

@boegelbot
Copy link
Collaborator

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

PR test command 'EB_PR=13139 EB_ARGS="--include-easyblocks-from-pr 2487" /apps/slurm/default/bin/sbatch --job-name test_PR_13139 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

- notification for comment with ID 902013446 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).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2487
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in total)
generoso-c1-s-5 - 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/d2da4b0d02c37ed04bdea5e8099c0eed for a full test report.

@easybuilders easybuilders deleted a comment from boegelbot Aug 19, 2021
boegel
boegel previously approved these changes Aug 19, 2021
@boegel
Copy link
Member

boegel commented Aug 19, 2021

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2487
SUCCESS
Build succeeded for 7 out of 7 (6 easyconfigs in total)
node3111.skitty.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/7adad6dba709de77ae3ddb954ef48a24 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 Aug 20, 2021

Going in, thanks @deniskristak!

@boegel boegel merged commit 0d57221 into easybuilders:develop Aug 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants