Skip to content

TensorBoard 2.11.1 #6141

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

Merged
merged 5 commits into from
Jan 13, 2023
Merged

TensorBoard 2.11.1 #6141

merged 5 commits into from
Jan 13, 2023

Conversation

arcra
Copy link
Member

@arcra arcra commented Jan 12, 2023

A patch release to replace the deprecated OOB auth flow, and remove deprecated aliases
that were removed in numpy 1.24.0.

Bug fixes

@arcra arcra requested a review from nfelt January 12, 2023 21:49
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.

Let's cherry-pick 2bfdca4 as well since it also addresses numpy 1.24.0 compatibility? Other than that and some release note wording comments, this looks great!

yatbear and others added 3 commits January 12, 2023 23:56
`numpy.bool8` is just a deprecated alias of
[`numpy.bool_`](https://numpy.org/doc/stable/reference/arrays.scalars.html#numpy.bool_).
Removed here to get rid of the deprecation warnings
(tensorflow#6110).

Googlers, see cl/498031924 for internal tests.

#oncall

(cherry picked from commit 2bfdca4)
Numpy library deprecated some type aliases in version 1.20.0 [1], and
then removed them in version 1.24.0 [2], which was released on Dec 18,
2022.

Without this change, our build would be broken when using numpy version
>= 1.24.0,
with error `AttributeError: module 'numpy' has no attribute 'float'`.

The fix suggested in release notes from numpy version 1.20.0 is to
replace these types with the equivalent primitive python types. (In this
case, simply `float`.)

[1]
http://numpy.org/doc/stable/release/1.20.0-notes.html#using-the-aliases-of-builtin-types-like-np-int-is-deprecated
[2]
http://numpy.org/doc/stable/release/1.24.0-notes.html#expired-deprecations

* Motivation for features / changes
Newer numpy versions break our build. This code is exactly equivalent,
as the identifiers used previously were aliases for the same type.

* Technical description of changes
Replace occurrences of `np.float` for the primitive type `float`.

* Screenshots of UI changes
N/A

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

* Alternate designs / implementations considered
N/A.

(cherry picked from commit 7bcc5e8)
…auth flow. (tensorflow#6107)

* Motivation for features / changes
The OOB auth flow has been deprecated. We concluded that the
limited-input device flow is appropriate for our use case where the
uploader runs in an environment where a browser is not available.

* Technical description of changes
Implements a new auth flow which calls an RPC to fetch a device_code,
verification_url and user_code, and asks user to visit the
verification_url in another device and enter the user_code; then starts
polling for the access token after the user authorizes the access from
another device.

* Screenshots of UI changes
N/A

* Detailed steps to verify changes work correctly (as executed by you)
- Wrote test script that uses this class, and tested the auth flow and
was able to print the credentials.
- Wrote tests.

* Alternate designs / implementations considered
Basically, implementing something similar to this flow or the OOB flow
ourselves.

(cherry picked from commit 8da06b5)
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.

LGTM to rebase-and-merge!

@arcra arcra merged commit 5ad9b7a into tensorflow:2.11 Jan 13, 2023
@arcra arcra deleted the 2.11 branch January 13, 2023 00:48
@arcra arcra restored the 2.11 branch January 13, 2023 00:49
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.

3 participants