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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants