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

tensorboard-data-server: add Linux aarch64 support #6101

Merged
merged 1 commit into from
Dec 12, 2022

Conversation

adamjstewart
Copy link
Contributor

  • Motivation for features / changes

When building tensorboard-data-server from source on Linux aarch64, pip install raises the following error:

pip._internal.exceptions.UnsupportedWheel: tensorboard_data_server-0.6.1-py3-none-manylinux2010_aarch64.whl is not a supported wheel on this platform.
  • Technical description of changes

From what I can tell, manylinux2014 is the first version that supports aarch64. TensorFlow also moved to manylinux2014, so there should be no issues with us updating as well: https://pypi.org/project/tensorflow/#files

  • Screenshots of UI changes

N/A

  • Detailed steps to verify changes work correctly (as executed by you)

I don't have access to Linux aarch64 myself (this was discovered in our CI) but anyone who does should be able to confirm that this works using:

$ cd tensorboard/data/server
$ cargo build --release
$ cd pip_package
$ python3 build.py
$ pip install *.whl
  • Alternate designs / implementations considered

Alternatively, we could use manylinux2010 on x86_64 and manylinux2014 on all other architectures, but I see no reason to avoid manylinux2014.

This is sort of a follow-up to #5715, @japie1235813 may be willing to review this

@bernhardkaindl
Copy link

@JamesHollyer Is this ok to merge?

@JamesHollyer
Copy link
Contributor

I think this update seems like a good idea. The tests failed but I think it might have actually been a flake. I am rerunning the job to find out.

@JamesHollyer JamesHollyer requested a review from bmd3k December 12, 2022 17:41
@bmd3k bmd3k requested review from groszewn and removed request for bmd3k December 12, 2022 18:00
Copy link
Contributor

@groszewn groszewn left a comment

Choose a reason for hiding this comment

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

Thanks for the update!

@groszewn groszewn merged commit 1675de9 into tensorflow:master Dec 12, 2022
@adamjstewart adamjstewart deleted the tb-data-server/aarch64 branch December 12, 2022 18:48
qihach64 pushed a commit to qihach64/tensorboard that referenced this pull request Dec 19, 2022
* Motivation for features / changes

When building tensorboard-data-server from source on Linux aarch64, pip
install raises the following error:
```
pip._internal.exceptions.UnsupportedWheel: tensorboard_data_server-0.6.1-py3-none-manylinux2010_aarch64.whl is not a supported wheel on this platform.
```

* Technical description of changes

From what I can tell, manylinux2014 is the first version that supports
aarch64. TensorFlow also moved to manylinux2014, so there should be no
issues with us updating as well:
https://pypi.org/project/tensorflow/#files

* Screenshots of UI changes

N/A

* Detailed steps to verify changes work correctly (as executed by you)

I don't have access to Linux aarch64 myself (this was discovered in our
CI) but anyone who does should be able to confirm that this works using:
```console
$ cd tensorboard/data/server
$ cargo build --release
$ cd pip_package
$ python3 build.py
$ pip install *.whl
```
* Alternate designs / implementations considered

Alternatively, we could use manylinux2010 on x86_64 and manylinux2014 on
all other architectures, but I see no reason to avoid manylinux2014.

This is sort of a follow-up to tensorflow#5715, @japie1235813 may be willing to
review this
yatbear pushed a commit to yatbear/tensorboard that referenced this pull request Mar 27, 2023
* Motivation for features / changes

When building tensorboard-data-server from source on Linux aarch64, pip
install raises the following error:
```
pip._internal.exceptions.UnsupportedWheel: tensorboard_data_server-0.6.1-py3-none-manylinux2010_aarch64.whl is not a supported wheel on this platform.
```

* Technical description of changes

From what I can tell, manylinux2014 is the first version that supports
aarch64. TensorFlow also moved to manylinux2014, so there should be no
issues with us updating as well:
https://pypi.org/project/tensorflow/#files

* Screenshots of UI changes

N/A

* Detailed steps to verify changes work correctly (as executed by you)

I don't have access to Linux aarch64 myself (this was discovered in our
CI) but anyone who does should be able to confirm that this works using:
```console
$ cd tensorboard/data/server
$ cargo build --release
$ cd pip_package
$ python3 build.py
$ pip install *.whl
```
* Alternate designs / implementations considered

Alternatively, we could use manylinux2010 on x86_64 and manylinux2014 on
all other architectures, but I see no reason to avoid manylinux2014.

This is sort of a follow-up to tensorflow#5715, @japie1235813 may be willing to
review this
dna2github pushed a commit to dna2fork/tensorboard that referenced this pull request May 1, 2023
* Motivation for features / changes

When building tensorboard-data-server from source on Linux aarch64, pip
install raises the following error:
```
pip._internal.exceptions.UnsupportedWheel: tensorboard_data_server-0.6.1-py3-none-manylinux2010_aarch64.whl is not a supported wheel on this platform.
```

* Technical description of changes

From what I can tell, manylinux2014 is the first version that supports
aarch64. TensorFlow also moved to manylinux2014, so there should be no
issues with us updating as well:
https://pypi.org/project/tensorflow/#files

* Screenshots of UI changes

N/A

* Detailed steps to verify changes work correctly (as executed by you)

I don't have access to Linux aarch64 myself (this was discovered in our
CI) but anyone who does should be able to confirm that this works using:
```console
$ cd tensorboard/data/server
$ cargo build --release
$ cd pip_package
$ python3 build.py
$ pip install *.whl
```
* Alternate designs / implementations considered

Alternatively, we could use manylinux2010 on x86_64 and manylinux2014 on
all other architectures, but I see no reason to avoid manylinux2014.

This is sort of a follow-up to tensorflow#5715, @japie1235813 may be willing to
review this
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.

4 participants