Skip to content

{vis}[foss/2022a] R-tesseract v5.1.0#17722

Merged
branfosj merged 1 commit intoeasybuilders:developfrom
smoors:20230414143148_new_pr_R-tesseract510
Apr 15, 2023
Merged

{vis}[foss/2022a] R-tesseract v5.1.0#17722
branfosj merged 1 commit intoeasybuilders:developfrom
smoors:20230414143148_new_pr_R-tesseract510

Conversation

@smoors
Copy link
Copy Markdown
Contributor

@smoors smoors commented Apr 14, 2023

(created using eb --new-pr)

depends on:

@smoors smoors added the update label Apr 14, 2023
@verdurin
Copy link
Copy Markdown
Member

@boegelbot please test @ jsc-zen2


name = 'R-tesseract'
version = '5.1.0'
versionsuffix = '-R-%(rver)s'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@smoors do we still need the versionsuffix?

Copy link
Copy Markdown
Contributor Author

@smoors smoors Apr 14, 2023

Choose a reason for hiding this comment

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

good question :)
a quick search gives me 6 easyconfigs without and 22 with the -R-4.2.1 versionsuffix..

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm happy to go ahead with this as it is, because:

  • we've got easyconfigs with and without the versionsuffix
  • we neither enforce that easyconfigs should be with or without versionsuffix

If we do want to do so then we can standardise them then.

@boegelbot
Copy link
Copy Markdown
Collaborator

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

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

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1508981082 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
Copy Markdown
Collaborator

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

@branfosj
Copy link
Copy Markdown
Member

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

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

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

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1509794090 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
Copy Markdown
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns1 - 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/977c11993f95868afb8f3f8548282a09 for a full test report.

@branfosj
Copy link
Copy Markdown
Member

Test report by @branfosj
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
bear-pg0105u03a.bear.cluster - Linux RHEL 8.6, x86_64, Intel(R) Xeon(R) Platinum 8360Y CPU @ 2.40GHz (icelake), Python 3.6.8
See https://gist.github.com/branfosj/87c679fe4aacdac7eb45ac08406b1f07 for a full test report.

@branfosj branfosj added this to the next release (4.7.2) milestone Apr 15, 2023
@branfosj
Copy link
Copy Markdown
Member

Going in, thanks @smoors!

@branfosj branfosj merged commit dde8622 into easybuilders:develop Apr 15, 2023
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.

4 participants