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

Set build type for PyTorch explicitely #3332

Merged
merged 3 commits into from
Jul 17, 2024

Conversation

Flamefire
Copy link
Contributor

PyTorch checks these environment variables in this order to determine the build type (for CMake) to use:

  • CMAKE_BUILD_TYPE
  • DEBUG
  • REL_WITH_DEB_INFO

If we don't set it explicitely we might end up with an unexpected debug build.
Add the build_typeEC option used in the CMakeMake easyblock with similar semantics.

PyTorch checks these environment variables in this order to determine
the build type (for CMake) to use:
- CMAKE_BUILD_TYPE
- DEBUG
- REL_WITH_DEB_INFO

If we don't set it explicitely we might end up with an unexpected debug
build.
Add the `build_type`EC option used in the CMakeMake easyblock with similar semantics.
@boegel boegel added this to the release after 4.9.1 milestone May 21, 2024
@boegel boegel modified the milestones: 4.9.2, release after 4.9.2 Jun 6, 2024
Co-authored-by: SebastianAchilles <s.achilles@fz-juelich.de>
@SebastianAchilles
Copy link
Member

Test report by @SebastianAchilles

Overview of tested easyconfigs (in order)

  • SUCCESS PyTorch-1.12.0-foss-2022a-CUDA-11.7.0.eb
  • SUCCESS PyTorch-2.1.2-foss-2022b.eb
  • SUCCESS PyTorch-1.12.0-foss-2022a.eb

Build succeeded for 3 out of 3 (3 easyconfigs in total)
bwd-rockylinux-94 - Linux Rocky Linux 9.4 (Blue Onyx), x86_64, Intel(R) Core(TM) i7-6900K CPU @ 3.20GHz (broadwell), 2 x NVIDIA NVIDIA GeForce GTX 1060 6GB, 535.161.08, Python 3.9.18
See https://gist.github.com/SebastianAchilles/6691cc73b7a496b696032d73a0e6a00d for a full test report.

@SebastianAchilles
Copy link
Member

Test report by @SebastianAchilles

Overview of tested easyconfigs (in order)

  • SUCCESS PyTorch-2.1.2-foss-2023a-CUDA-12.1.1.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
skl-rockylinux-810 - Linux Rocky Linux 8.10, x86_64, Intel(R) Core(TM) i7-10700 CPU @ 2.90GHz (skylake), 1 x NVIDIA NVIDIA RTX A4000, 555.42.02, Python 3.6.8
See https://gist.github.com/SebastianAchilles/e350ef77f6ffb4691339c118ebd2a87d for a full test report.

Copy link
Member

@ocaisa ocaisa left a comment

Choose a reason for hiding this comment

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

LGTM, and tests are in

@ocaisa ocaisa merged commit 0e6e2bb into easybuilders:develop Jul 17, 2024
41 checks passed
@Flamefire Flamefire deleted the fix-pytorch-build-type branch July 17, 2024 08:54
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