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/2022a] QIIME2 v2023.5.1, UniFrac v1.3.2 #18407

Merged

Conversation

smoors
Copy link
Contributor

@smoors smoors commented Jul 27, 2023

(created using eb --new-pr)
depends on:

based on #16945 (@migueldiascosta )

…-2022a.eb and patches: QIIME2-2023.5.1_fix.patch, unifrac-1.1.1_fix.patch, UniFrac-1.3.2_fix.patch
@verdurin
Copy link
Member

@boegelbot please test @ generoso

@verdurin
Copy link
Member

@smoors ah, I missed the Seaborn dep. Is that in progress, or something you need to add?

@boegelbot
Copy link
Collaborator

@verdurin: Request for testing this PR well received on login1

PR test command 'EB_PR=18407 EB_ARGS= EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_18407 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

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

@smoors
Copy link
Contributor Author

smoors commented Jul 28, 2023

@smoors ah, I missed the Seaborn dep. Is that in progress, or something you need to add?

@verdurin my bad, we accidentally have a different version installed in our cluster.
fixed.

@casparvl
Copy link
Contributor

casparvl commented Aug 1, 2023

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

@casparvl: Request for testing this PR well received on login1

PR test command 'EB_PR=18407 EB_ARGS= EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_18407 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

- notification for comment with ID 1659892979 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
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
cns3 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/boegelbot/1ad144e2537952644cc732cb3ef3cd56 for a full test report.

@casparvl
Copy link
Contributor

casparvl commented Aug 1, 2023

@boegelbot please test @ jsc-zen2

@boegelbot
Copy link
Collaborator

@casparvl: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster

PR test command 'EB_PR=18407 EB_ARGS= /opt/software/slurm/bin/sbatch --mem-per-cpu=4000M --job-name test_PR_18407 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen2.sh' executed!

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

Test results coming soon (I hope)...

- notification for comment with ID 1659955674 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
SUCCESS
Build succeeded for 3 out of 3 (2 easyconfigs in total)
jsczen2g1.int.jsc-zen2.easybuild-test.cluster - Linux Rocky Linux 8.5, x86_64, AMD EPYC 7742 64-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/boegelbot/4e51084704a374f31a387d757736d3e5 for a full test report.

@casparvl
Copy link
Contributor

casparvl commented Aug 2, 2023

Test report by @casparvl
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
tcn222.local.snellius.surf.nl - Linux RHEL 8.6, x86_64, AMD EPYC 7H12 64-Core Processor, Python 3.6.8
See https://gist.github.com/casparvl/2158a8c8ce098731b5952773fd5cf26d for a full test report.

Copy link
Contributor

@casparvl casparvl 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 to me. Just for future reference: this QIIME2 easyconfig is the first that isn't based on a conda easyblock (as previous versions where). That's why it is now at the foss level and dependent on a large quantity of dependencies.

I applaud the effort @smoors , this must have been quite a bit of work!

@casparvl casparvl added this to the next release (4.8.1?) milestone Aug 2, 2023
@casparvl
Copy link
Contributor

casparvl commented Aug 2, 2023

Going in, thanks @smoors!

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.

4 participants