Skip to content

Force old version of numpy in noise.ipynb#740

Closed
markmcd wants to merge 1 commit into
tensorflow:masterfrom
markmcd:patch-1
Closed

Force old version of numpy in noise.ipynb#740
markmcd wants to merge 1 commit into
tensorflow:masterfrom
markmcd:patch-1

Conversation

@markmcd
Copy link
Copy Markdown
Member

@markmcd markmcd commented Jan 19, 2023

This currently works in colab, but will break when it's updated, and breaks in the TF docs publishing infra.

The TFQ code is referencing NumPy's np.float, which is removed in v1.24. Details here. This is quick hack to get the docs working again.

This currently works in colab, but will break when it's updated, and breaks in the TF docs publishing infra.

The TFQ code is referencing NumPy's `np.float`, which is removed in v1.24. [Details here](https://numpy.org/doc/stable/release/1.20.0-notes.html#using-the-aliases-of-builtin-types-like-np-int-is-deprecated). This is quick hack to get the docs working again.
@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@jaeyoo
Copy link
Copy Markdown
Member

jaeyoo commented Mar 20, 2023

#749 solves this PR together. closed.

@jaeyoo jaeyoo closed this Mar 20, 2023
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