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.

Sign up for GitHub

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

Jump to bottom

Fix PYTHONPATH for hermetic python in TensorFlow builds with EB 5.x #3568

Open
wants to merge 1 commit into
base: 5.0.x
Choose a base branch
from
Open

Fix PYTHONPATH for hermetic python in TensorFlow builds with EB 5.x #3568

wants to merge 1 commit into from

Conversation

yqshao
Copy link
Contributor

@yqshao yqshao commented Jan 27, 2025 •
edited
Loading

Trying to fix #3566

@yqshao
Copy link
Contributor Author

yqshao commented Jan 27, 2025 •
edited
Loading

While testing this I also realized that the -Og flag no longer works for TF>2.14 as a patch fixing it was rolled-back because that conflicts with some JAX CI (which seeems to be running python2).

Bringing that patch back does not seem to trivially fix our debug=True build, currently I'm working around this by disabling debug at our site, but not sure what's the best way to handle it in EB.

@yqshao yqshao changed the title Fix PYTHONPATH for hermetic python in TF builds with EB 5.x Fix PYTHONPATH for hermetic python in TensorFlow builds with EB 5.x Jan 28, 2025
@boegel boegel requested a review from Micket January 29, 2025 08:47
@boegel boegel added bug fix EasyBuild-5.0 EasyBuild 5.0 labels Jan 29, 2025
@boegel boegel added this to the 5.0 milestone Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Micket Micket Awaiting requested review from Micket

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
bug fix EasyBuild-5.0 EasyBuild 5.0
Projects
EasyBuild v5.0
Status: Breaking changes
Milestone
5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@yqshao @boegel