Skip to content

Conversation

@yashk2810
Copy link
Member

Utils contains types/ and since the API generator now supports type annotation with links, it will be good to link off to the type pages.

Utils contains types/ and since the API generator now supports type annotation with links, it will be good to link off to the type pages.
@MarkDaoust
Copy link
Member

Would it make sense to import that utils.types in the root init.py so you don't document the rest of utils too?

Or switch to that explicit package thing and add import tensorflow_addons.utils.types to utils?

Can you post a diff of the output TOC either way?

@gabrieldemarmiesse
Copy link
Member

Maybe you'd need to bump the version for the CI to pass? See https://github.com/tensorflow/addons/blob/master/tools/install_deps/doc_requirements.txt

@facaiy
Copy link
Member

facaiy commented Jun 3, 2020

I think utils module is hidden intentionally, because we don't want to guarantee its backward compatibility.

@WindQAQ WindQAQ closed this Dec 18, 2020
@WindQAQ
Copy link
Member

WindQAQ commented Dec 18, 2020

Should be done in #2162

@seanpmorgan seanpmorgan deleted the yashk2810-patch-1 branch December 18, 2020 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants