Skip to content

Conversation

@harishsdev
Copy link
Contributor

Example code for MuLawDecoding Transform

@harishsdev
Copy link
Contributor Author

@nateanl please he to review this PR

Copy link
Member

@nateanl nateanl left a comment

Choose a reason for hiding this comment

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

LGTM. Just need to fix the nit.

Args:
quantization_channels (int, optional): Number of channels. (Default: ``256``)
Example
Copy link
Member

Choose a reason for hiding this comment

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

Let's add a line before Example

Suggested change
Example
Example

@nateanl
Copy link
Member

nateanl commented Aug 6, 2021

Could you also clean the commit history in the PR? You can do it by rebase to the main branch and force push it.

@harishsdev harishsdev requested a review from nateanl August 6, 2021 10:23
Copy link
Member

@nateanl nateanl left a comment

Choose a reason for hiding this comment

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

LGTM. Next time when you create the PR, could you reset your local branch to the latest main branch? So that we can have only the related commits in the commit history. Thanks! @harishsdev

@nateanl nateanl merged commit 90c0edc into pytorch:main Aug 6, 2021
@harishsdev
Copy link
Contributor Author

harishsdev commented Aug 6, 2021

git fetch origin
git reset --hard origin/master

above commands are correct,i will reset from next onwards
Thanks @nateanl

@nateanl nateanl mentioned this pull request Aug 6, 2021
19 tasks
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.

3 participants