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

chore: Upgrade html5lib to version 1.1. #5490

Merged
merged 2 commits into from
Jan 11, 2022

Conversation

bmd3k
Copy link
Contributor

@bmd3k bmd3k commented Jan 7, 2022

As noted by #5478 , tensorboard fails when run in a python 3.10 environment. The reporter fortunately discovered that the problems would be fixed with an upgrade of our html5lib dependency.

I tested by building a python3.10 virtualenv on my machine (this required building python 3.10 from source as we don't have it natively).

$ virtualenv -p /usr/local/bin/python3.10 ~/virtualenv/tensorboard-python-3.10
$ source ~/virtualenv/tensorboard-python-3.10/bin/activate
$ python3.10 -m ensurepip --upgrade
$ python3.10 -m pip install tensorboard
$ tensorboard --version
Traceback (most recent call last):
  File "/usr/local/google/home/bdubois/virtualenv/tensorboard-python-3.10/bin/tensorboard", line 5, in <module>
    from tensorboard.main import run_main
<snip>
    from collections import Mapping
ImportError: cannot import name 'Mapping' from 'collections' (/usr/local/lib/python3.10/collections/__init__.py)
$ python3.10 -m pip install --upgrade /tmp/tb-with-html5lib-1.1/tensorboard*.whl
$ tensorboard --version
TensorFlow installation not found - running with reduced feature set.
2.8.0a0

I also ran the tensorboard with some logdirs and did some basic sanity. I specifically checked that markdown rendering seemed to be working for a couple text plugin logs that I have.

@bmd3k bmd3k requested a review from nfelt January 7, 2022 20:11
Copy link
Contributor

@nfelt nfelt 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 taking care of this!

I chose to use the PyPi URL for the download of the dependency as I had difficulties finding the sha256 hashes for the version of the library at mirror.tensorflow.org and github.com (they differ from the PyPi one).

It's fine to just generate the sha256 hash locally - i.e. download the tarball and compute the hash yourself. In this case it might be easier to still do that, since the github one is already mirrored (see other comment).

FWIW, to justify that a little - the point of the hash isn't really to verify the integrity of the file that you're downloading today from the server, because in general we assume you're downloading it via HTTPS, on a reasonably secure machine and network, etc, and it doesn't really add much protection to separately get a hash from the same server rather than just computing it yourself. The point is to ensure the server doesn't change the contents of the file in the future. That way, we reduce the amount of trust we need to have from "I trust this server in perpetuity" to "I trust this server right now, today".

third_party/python.bzl Show resolved Hide resolved
@bmd3k bmd3k requested a review from nfelt January 10, 2022 15:27
@bmd3k bmd3k merged commit ae19add into tensorflow:master Jan 11, 2022
bmd3k added a commit to bmd3k/tensorboard that referenced this pull request Jan 19, 2022
As noted by tensorflow#5478 , tensorboard fails when run in a python 3.10 environment. The reporter fortunately discovered that the problems would be fixed with an upgrade of our html5lib dependency.

I tested by building a python3.10 virtualenv on my machine (this required building python 3.10 from source as we don't have it natively).

I also ran the tensorboard with some logdirs and did some basic sanity. I specifically checked that markdown rendering seemed to be working for a couple text plugin logs that I have.
bmd3k added a commit that referenced this pull request Jan 20, 2022
As noted by #5478 , tensorboard fails when run in a python 3.10 environment. The reporter fortunately discovered that the problems would be fixed with an upgrade of our html5lib dependency.

I tested by building a python3.10 virtualenv on my machine (this required building python 3.10 from source as we don't have it natively).

I also ran the tensorboard with some logdirs and did some basic sanity. I specifically checked that markdown rendering seemed to be working for a couple text plugin logs that I have.
yatbear pushed a commit to yatbear/tensorboard that referenced this pull request Mar 27, 2023
As noted by tensorflow#5478 , tensorboard fails when run in a python 3.10 environment. The reporter fortunately discovered that the problems would be fixed with an upgrade of our html5lib dependency.

I tested by building a python3.10 virtualenv on my machine (this required building python 3.10 from source as we don't have it natively).

I also ran the tensorboard with some logdirs and did some basic sanity. I specifically checked that markdown rendering seemed to be working for a couple text plugin logs that I have.
dna2github pushed a commit to dna2fork/tensorboard that referenced this pull request May 1, 2023
As noted by tensorflow#5478 , tensorboard fails when run in a python 3.10 environment. The reporter fortunately discovered that the problems would be fixed with an upgrade of our html5lib dependency.

I tested by building a python3.10 virtualenv on my machine (this required building python 3.10 from source as we don't have it natively).

I also ran the tensorboard with some logdirs and did some basic sanity. I specifically checked that markdown rendering seemed to be working for a couple text plugin logs that I have.
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.

2 participants