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

Make tensorflow profiler into OpenXLA #22932

Open
yliu120 opened this issue Feb 21, 2025 · 3 comments
Open

Make tensorflow profiler into OpenXLA #22932

yliu120 opened this issue Feb 21, 2025 · 3 comments
Labels
question Further information is requested

Comments

@yliu120
Copy link
Contributor

yliu120 commented Feb 21, 2025

Hi,

I assume that many users in the community has the same issue that we need to pip install tensorflow even though we don't use tensorflow. The reason for that is that the tensorboard depends on tensorflow.

I guess only some very small part of the profiler frontend has some dependencies on the util libraries inside tensorflow. Can we make tensorboard depending purely on XLA so that all JAX users don't have to install tensorflow to use the profiler tool?

The reasons why we don't like the tensorflow package:

  1. Very very big, ~1.5GiB in size and we don't use it.
  2. depends on very old protobuf library which gives us many dependency conflicts.

I see two directions:

  1. in a short term, I guess we can potentially update the protobuf library of XLA to some latest version.
  2. in the long run, we should completely lift tensorboard from TF code base.
@yliu120 yliu120 changed the title Tensorflow profiler into OpenXLA Make tensorflow profiler into OpenXLA Feb 21, 2025
@cliveverghese
Copy link
Contributor

Hi,

With version 2.19.0 of the profiler, You don't have to install tensorflow.

Just the following should be sufficient.

pip install tensorboard tensorboard-plugin-profile

@yliu120
Copy link
Contributor Author

yliu120 commented Feb 26, 2025

I have an additional question:

https://github.com/tensorflow/tensorboard/blob/master/WORKSPACE#L147-L155

If we don't constraint on being compatible with TF, we don't need to pin protobuf version to 3.x?

@aniruthraj aniruthraj added the question Further information is requested label Mar 13, 2025
@Matt-Hurd
Copy link
Contributor

While we don't require tensorflow, there are still codepaths that do use it. Example. If all of these are removed, then we can do a pass over our dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants