Skip to content

{tools}[foss/2020a] GetOrganelle v1.7.2 w/ Python 3.8.2#11948

Merged
boegel merged 5 commits intoeasybuilders:developfrom
deniskristak:20210104145805_new_pr_GetOrganelle172
Jan 6, 2021
Merged

{tools}[foss/2020a] GetOrganelle v1.7.2 w/ Python 3.8.2#11948
boegel merged 5 commits intoeasybuilders:developfrom
deniskristak:20210104145805_new_pr_GetOrganelle172

Conversation

@deniskristak
Copy link
Copy Markdown
Contributor

(created using eb --new-pr)

@deniskristak
Copy link
Copy Markdown
Contributor Author

Copy link
Copy Markdown
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.

@deniskristak Please also set use_pip and sanity_pip_check to True

@easybuilders easybuilders deleted a comment from boegelbot Jan 4, 2021
@easybuilders easybuilders deleted a comment from boegelbot Jan 4, 2021
Copy link
Copy Markdown
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.

@deniskristak A couple more things to change:

  • add sanity_pip_check = True (see failing tests);
  • add fix_python_shebang_for = ['bin/*.py'] to fix the shebang line in the Python script to not hardcode the path to the python command, but use /usr/bin/env python instead;
  • add a custom sanity_check_paths which also checks for a couple of the scripts:
sanity_check_paths = {
    'files': ['bin/check_annotations.py', 'bin/get_organelle_from_reads.py', 'bin/slim_graph.py'],
    'dirs': ['lib/python%(pyshortver)s/site-packages'],
}

@easybuilders easybuilders deleted a comment from boegelbot Jan 6, 2021
@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2021

@boegelbot please test @ generoso

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2021

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

@boegelbot
Copy link
Copy Markdown
Collaborator

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

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

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

Test results coming soon (I hope)...

Details

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2021

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node2608.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/475d0948e74508af58b8d3453b817000 for a full test report.

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 2 out of 2 (1 easyconfigs in total)
generoso-x-2 - 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/0acd507a4f2c3dc6aac81c3f07fbcdd3 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2021

Test report by @boegel
SUCCESS
Build succeeded for 3 out of 3 (1 easyconfigs in total)
node3106.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/24eb97be8224b715329352ca3501d5e6 for a full test report.

@boegel boegel added the new label Jan 6, 2021
@boegel boegel added this to the next release (4.3.3?) milestone Jan 6, 2021
@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2021

Going in, thanks @deniskristak!

@boegel boegel merged commit 28824cd into easybuilders:develop Jan 6, 2021
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