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

find rocm components individually #2566

Closed

Conversation

afzpatel
Copy link

Required for spack since all the rocm components are located in different paths

tensorflow-jenkins and others added 30 commits August 3, 2023 14:45
* Insert release notes place-fill

* Update RELEASE.md

---------

Co-authored-by: Raviteja Gorijala <36429068+rtg0795@users.noreply.github.com>
…umbers-2.14.0rc0-12810

Update version numbers for TensorFlow 2.14.0-rc0
r2.14 cherry-pick: 63d1596 "version bump and formatting"
Update setup.py on TF release branch with released version
XLA unit tests were unable to link due to undefined reference to
stream_executor::Stream::BlockHostUntilDone()
r2.14 cherry-pick: de4a9a8 "Fix link failure in XLA unit tests"
Update requirements.in to bump keras-nightly and tf-estimator-nightly
Run requirements updater to bump keras, tb and estimator
PiperOrigin-RevId: 554977992

Co-authored-by: Adrian Revuelta <adrianrc@google.com>
Revert "Update Windows Release Build File"
…umbers-2.14.0rc1-30454

Update version numbers for TensorFlow 2.14.0-rc1
Update requirements_lock.txt files
…corrupted PT_NOTE program headers.

It seems that using Patchelf <0.12 during the [wheel building process](https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/pip_package/build_pip_package.sh#L255-L262) corrupts the PT_NOTE program headers of certain shared libraries. Upgrading the patchelf version seems to fix this. We need to download the latest version from GitHub because the default Ubuntu Focal packages only provide the "0.10-2build1" version.

PiperOrigin-RevId: 560754278
It has been deprecated since 2019, and is now triggering warnings on
TensorFlow startup.  It does, however, still need to be included in
`python/__init__.py` so that it can be found by `tf.compat.v1.distributions`,
which several projects depend on.

We will remove the entire package from TensorFlow as a follow-up.  But
at the moment, doing so would break too many other projects.

PiperOrigin-RevId: 558816905
draganmladjenovic and others added 23 commits March 15, 2024 15:55
[r2.14-rocm-enhanced][ROCm] Allow XLA LLVM to read rocm 6.2 'libdevice' bitcode files
…wd (ROCm#2474)

This change is helping to permanently get rid of below issue.

02:56:54 Step 29/38 : RUN echo 'ALL ALL=NOPASSWD:ALL' | tee /etc/sudoers.d/sudo-nopasswd
02:56:54 ---> Running in f871d0cf3b96
02:56:54 �[91mtee: /etc/sudoers.d/sudo-nopasswd: No such file or directory
02:56:54 �[0mALL ALL=NOPASSWD:ALL
02:56:54 The command '/bin/sh -c echo 'ALL ALL=NOPASSWD:ALL' | tee /etc/sudoers.d/sudo-nopasswd' returned a non-zero code: 1
…_fix

[r2.14-rocm-enhanced][ROCM] adding env variable to control rocm_dnn workspace limit
[r2.14-rocm-enhanced] Include numpy WAR in run_cpu.sh script
[ROCM][R2.14-rocm-enhanced] Workaround for TF issue 62466
[r2.14-rocm-enhanced] Adjust rocm-smi check for test scripts
r2.14-rocm-enhanced - skipping mgpu test in sgpu script
[r2.14-rocm-enhanced] Update 2.14 branch to 2.14.1
[r2.14-rocm-enhanced] Fix a typo in setup.rocm.cs7.sh
@afzpatel afzpatel requested review from i-chaochen and hsharsha June 10, 2024 16:13
@hsharsha
Copy link

@afzpatel Can you please add this to develop-upstream and trigger CI. Once CI passes we can backport that change to all the other required branches.

@afzpatel afzpatel changed the base branch from r2.14-rocm-enhanced-spack to develop-upstream June 10, 2024 18:33
@afzpatel
Copy link
Author

woops changing the base got the pr really messy, gonna close this and open a new one

@afzpatel afzpatel closed this Jun 10, 2024
@afzpatel afzpatel deleted the find-rocm-components-individually branch June 10, 2024 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.