Thanks to visit codestin.com
Credit goes to github.com

Skip to content

chore: Upgrade html5lib to version 1.1.#5490

Merged
bmd3k merged 2 commits into
tensorflow:masterfrom
bmd3k:html5lib-upgrade
Jan 11, 2022
Merged

chore: Upgrade html5lib to version 1.1.#5490
bmd3k merged 2 commits into
tensorflow:masterfrom
bmd3k:html5lib-upgrade

Conversation

@bmd3k
Copy link
Copy Markdown
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
Copy Markdown
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".

Comment thread third_party/python.bzl
@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