Skip to content

{vis}[foss/2022a] wxPython v4.2.1#18427

Merged
casparvl merged 6 commits intoeasybuilders:developfrom
ThomasHoffmann77:20230731173709_new_pr_attrdict3202
Aug 4, 2023
Merged

{vis}[foss/2022a] wxPython v4.2.1#18427
casparvl merged 6 commits intoeasybuilders:developfrom
ThomasHoffmann77:20230731173709_new_pr_attrdict3202

Conversation

@ThomasHoffmann77
Copy link
Copy Markdown
Contributor

(created using eb --new-pr)

@boegelbot
Copy link
Copy Markdown
Collaborator

@ThomasHoffmann77: Tests failed in GitHub Actions, see https://github.com/easybuilders/easybuild-easyconfigs/actions/runs/5717028706
Output from first failing test suite run:

FAIL: test_dep_versions_per_toolchain_generation (test.easyconfigs.easyconfigs.EasyConfigTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/easybuild-easyconfigs/easybuild-easyconfigs/test/easyconfigs/easyconfigs.py", line 877, in test_dep_versions_per_toolchain_generation
    self.assertFalse(multi_dep_vars, error_msg)
AssertionError: ['Wayland', 'glew'] is not false : No multi-variant deps found for '^.*-(?P<tc_gen>20(1[89]|[2-9][0-9])[ab]).*\.eb$' easyconfigs:

found 2 variants of 'Wayland' dependency in easyconfigs using '2022a' toolchain generation
* version: 1.20.0; versionsuffix:  as dep for {'segment-anything-1.0-foss-2022a.eb', 'Xmipp-3.22.07-Helios-foss-2022a.eb', 'LayoutParser-0.3.4-foss-2022a.eb', 'Ultralytics-8.0.92-foss-2022a-CUDA-11.7.0.eb', 'Albumentations-1.3.0-foss-2022a.eb', 'OpenCV-4.6.0-foss-2022a-CUDA-11.7.0-contrib.eb', 'MONAI-Label-0.5.2-foss-2022a-PyTorch-1.12.0-CUDA-11.7.0.eb', 'OpenCV-4.6.0-foss-2022a-contrib.eb', 'imgaug-0.4.0-foss-2022a-CUDA-11.7.0.eb', 'ASAP-2.1-foss-2022a.eb', 'pysteps-1.7.1-foss-2022a.eb', 'imgaug-0.4.0-foss-2022a.eb', 'MONAI-Label-0.5.2-foss-2022a-PyTorch-1.12.0.eb', 'LayoutParser-0.3.4-foss-2022a-CUDA-11.7.0.eb', 'Cellpose-2.2.2-foss-2022a-CUDA-11.7.0.eb', 'Albumentations-1.3.0-foss-2022a-CUDA-11.7.0.eb', 'Xmipp-3.22.07-Helios-foss-2022a-CUDA-11.7.0.eb', 'Cellpose-2.2.2-foss-2022a.eb'}
* version: 1.21.0; versionsuffix:  as dep for {'wxPython-4.2.1-foss-2022a.eb'}

found 2 variants of 'glew' dependency in easyconfigs using '2022a' toolchain generation
* version: 2.2.0; versionsuffix: -egl as dep for {'wxPython-4.2.1-foss-2022a.eb'}
* version: 2.2.0; versionsuffix: -osmesa as dep for {'python-mujoco-2.2.2-foss-2022a.eb'}


----------------------------------------------------------------------
Ran 17638 tests in 1033.562s

FAILED (failures=1)
ERROR: Not all tests were successful

bleep, bloop, I'm just a bot (boegelbot v20200716.01)
Please talk to my owner @boegel if you notice me acting stupid),
or submit a pull request to https://github.com/boegel/boegelbot fix the problem.

Wayland 1.21.0 -> 1.20.0
allow variant glew-2.2.0-foss-2022a-egl for wxPython-4.2.0-foss-2022a
adapt Wayland dependency from version 1.21.0 to 1.20.0
add glew 2.2.0 egl multi-variant deps: libwpe-1.14.1-GCCcore-11.3.0.eb, wpebackend-fdo-1.14.1-GCCcore-11.3.0.eb
@casparvl
Copy link
Copy Markdown
Contributor

casparvl commented Aug 1, 2023

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

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

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

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1660128074 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 9 out of 9 (9 easyconfigs in total)
cns2 - 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/fb27bd3aba3e355f6aec86bc3d478304 for a full test report.

@casparvl
Copy link
Copy Markdown
Contributor

casparvl commented Aug 1, 2023

@boegelbot please test @ jsc-zen2

@boegelbot
Copy link
Copy Markdown
Collaborator

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

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

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1660586043 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 9 out of 9 (9 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/48b5df96c4fe9e6b5a09c33e8edf809c for a full test report.

@casparvl
Copy link
Copy Markdown
Contributor

casparvl commented Aug 3, 2023

Test report by @casparvl
FAILED
Build succeeded for 8 out of 9 (9 easyconfigs in total)
tcn30.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/233a55395dc02f25c6ba10405d5e6f91 for a full test report.

@ThomasHoffmann77
Copy link
Copy Markdown
Contributor Author

Test report by @casparvl FAILED Build succeeded for 8 out of 9 (9 easyconfigs in total) tcn30.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/233a55395dc02f25c6ba10405d5e6f91 for a full test report.

is the failure due to "_scratch-nvme_1_casparl_generic_software_WebKitGTK+_2.40.4-GCC-11.3.0.lock already exists", or do I need to fix something else? @casparvl

@casparvl
Copy link
Copy Markdown
Contributor

casparvl commented Aug 4, 2023

@ThomasHoffmann77 yeah, sorry, I had a build that was broken off prematurely (job got killed due to walltime), but forgot to remove the lock file. It's fine: the module built at some point, even if that build did not upload a test report. Plus, there are the two boegelbot builds that were also succesful. Enough to convince me, I'll approve the PR :)

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

Lgtm!

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

casparvl commented Aug 4, 2023

Going in, thanks @ThomasHoffmann77!

@casparvl casparvl merged commit 76a6428 into easybuilders:develop Aug 4, 2023
@ThomasHoffmann77 ThomasHoffmann77 deleted the 20230731173709_new_pr_attrdict3202 branch October 19, 2023 13:13
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