Skip to content

replace run_cmd with run_shell_cmd in custom easyblock for tensorflow_compression (tensorflow_compression.py)#3164

Merged
boegel merged 1 commit intoeasybuilders:5.0.xfrom
branfosj:20240207115254_new_pr_tensorflow_compression
Feb 11, 2024
Merged

replace run_cmd with run_shell_cmd in custom easyblock for tensorflow_compression (tensorflow_compression.py)#3164
boegel merged 1 commit intoeasybuilders:5.0.xfrom
branfosj:20240207115254_new_pr_tensorflow_compression

Conversation

@branfosj
Copy link
Copy Markdown
Member

@branfosj branfosj commented Feb 7, 2024

(created using eb --new-pr)

  • no use of EB_tensorflow_minus_compression in easyconfigs or blocks

…flow_compression (`tensorflow_compression.py`)
@branfosj
Copy link
Copy Markdown
Member Author

branfosj commented Feb 7, 2024

Test report by @branfosj

Overview of tested easyconfigs (in order)

  • SUCCESS tensorflow-compression-2.11.0-foss-2022a-CUDA-11.7.0.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
bear-pg0208u15a - Linux RHEL 8.6, x86_64, Intel(R) Xeon(R) Platinum 8360Y CPU @ 2.40GHz (icelake), 1 x NVIDIA NVIDIA A100-SXM4-40GB, 520.61.05, Python 3.6.8
See https://gist.github.com/branfosj/da6c13fd86c979c785be3a8bc234be63 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 11, 2024

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS tensorflow-compression-2.11.0-foss-2022a-CUDA-11.7.0.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3301.joltik.os - Linux RHEL 8.8, x86_64, Intel(R) Xeon(R) Gold 6242 CPU @ 2.80GHz, 1 x NVIDIA Tesla V100-SXM2-32GB, 535.154.05, Python 3.6.8
See https://gist.github.com/boegel/8ecc1b87cc2227f6a9b17f154d861250 for a full test report.

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.

lgtm

@boegel boegel merged commit 4c26dc6 into easybuilders:5.0.x Feb 11, 2024
@branfosj branfosj deleted the 20240207115254_new_pr_tensorflow_compression branch February 11, 2024 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants