Skip to content

{phys}[GCCcore/12.3.0] PyAEDT v0.8.7#20357

Merged
Micket merged 3 commits intoeasybuilders:developfrom
appolloford:20240412213154_new_pr_PyAEDT087
Apr 24, 2024
Merged

{phys}[GCCcore/12.3.0] PyAEDT v0.8.7#20357
Micket merged 3 commits intoeasybuilders:developfrom
appolloford:20240412213154_new_pr_PyAEDT087

Conversation

@appolloford
Copy link
Copy Markdown
Contributor

@appolloford appolloford commented Apr 12, 2024

(created using eb --new-pr)

@appolloford
Copy link
Copy Markdown
Contributor Author

Test report by @appolloford
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
vera-skylake-build - Linux Rocky Linux 8.9, x86_64, Intel Xeon Processor (Skylake, IBRS, no TSX), Python 3.6.8
See https://gist.github.com/appolloford/b0107145976ed54f2d50986f534c7b29 for a full test report.

@jfgrimm
Copy link
Copy Markdown
Member

jfgrimm commented Apr 13, 2024

hmm, the test failure is down to the test suite not recognising that one is a wrapper

AssertionError: Should not have multiple variants of dependencies.
Found 2 variants of 'dotNET-Core' dependency in easyconfigs using 'GCCcore-12.3' toolchain generation

  • version: 8.0.203; versionsuffix: as dep for {'PyAEDT-0.8.7-GCCcore-12.3.0.eb'}
  • version: 8.0; versionsuffix: as dep for {'PyAEDT-0.8.7-GCCcore-12.3.0.eb'}

@migueldiascosta migueldiascosta added this to the 4.x milestone Apr 16, 2024
@appolloford appolloford force-pushed the 20240412213154_new_pr_PyAEDT087 branch from 052c22e to f938cf7 Compare April 18, 2024 16:40
@Micket
Copy link
Copy Markdown
Contributor

Micket commented Apr 18, 2024

I asked @appolloford to also update these tests to cover the net dotNET-Core, which should be treated identically to the Java wrappers (since they are).

@Micket Micket modified the milestones: 4.x, release after 4.9.1 Apr 18, 2024
@jfgrimm
Copy link
Copy Markdown
Member

jfgrimm commented Apr 18, 2024

@boegelbot please test @ generoso

Comment thread easybuild/easyconfigs/p/PyAEDT/PyAEDT-0.8.7-GCCcore-12.3.0.eb Outdated
@boegelbot
Copy link
Copy Markdown
Collaborator

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

PR test command 'EB_PR=20357 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs /opt/software/slurm/bin/sbatch --job-name test_PR_20357 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 2064578004 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.9, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/boegelbot/2e4abcdee58a9db90df455a7724178db for a full test report.

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Apr 22, 2024

Test report by @Micket
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
vera-skylake-build - Linux Rocky Linux 8.9, x86_64, Intel Xeon Processor (Skylake, IBRS, no TSX), Python 3.6.8
See https://gist.github.com/Micket/c04b7bd46947d66e86508480b9b5ec35 for a full test report.

Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

lgtm

@Micket Micket merged commit 04f769d into easybuilders:develop Apr 24, 2024
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.

5 participants