Skip to content

Conversation

@vaharoni
Copy link
Contributor

In the Quickstart for experts tutorial there is the following code:

class MyModel(Model): def __init__(self): super(MyModel, self).__init__() # ...

I suggest to use super().__init__() instead as described everywhere else.

@vaharoni vaharoni requested a review from a team as a code owner August 27, 2023 07:27
@github-actions
Copy link

Preview

Preview and run these notebook edits with Google Colab: Rendered notebook diffs available on ReviewNB.com.

Format and style

Use the TensorFlow docs notebook tools to format for consistent source diffs and lint for style:
$ python3 -m pip install -U --user git+https://github.com/tensorflow/docs 
$ python3 -m tensorflow_docs.tools.nbfmt notebook.ipynb
$ python3 -m tensorflow_docs.tools.nblint --arg=repo:tensorflow/docs notebook.ipynb
If commits are added to the pull request, synchronize your local branch: git pull origin tutorial_quickstart_expert_fix
@8bitmp3 8bitmp3 added the review in progress Someone is actively reviewing this PR label Aug 29, 2023
Copy link
Member

@MarkDaoust MarkDaoust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. Thanks!

@MarkDaoust MarkDaoust added ready to pull Start merge process and removed review in progress Someone is actively reviewing this PR labels Aug 30, 2023
@github-actions github-actions bot added the lgtm Community-added approval label Aug 30, 2023
@copybara-service copybara-service bot merged commit 96c420c into tensorflow:master Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Community-added approval ready to pull Start merge process

4 participants