Skip to content

Conversation

@lara-hdr
Copy link
Contributor

@lara-hdr lara-hdr commented Mar 5, 2019

No description provided.

@ezyang ezyang requested a review from houseroad March 6, 2019 18:15
@houseroad
Copy link
Member

@lara-hdr to pass the CI, you need to rebase to a newer commit on master

Copy link
Member

@houseroad houseroad left a comment

Choose a reason for hiding this comment

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

Looks good, but please rebase the PR, and trigger the CI again.

@ezyang
Copy link
Contributor

ezyang commented Mar 11, 2019

@pytorchbot rebase this please

@houseroad
Copy link
Member

Oh, this is a nice feature !

@lara-hdr
Copy link
Contributor Author

@houseroad all tests are passing with the rebase

@lara-hdr
Copy link
Contributor Author

@houseroad , can we merge this? :)

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@houseroad has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Member

@houseroad houseroad left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks!

@zrphercule zrphercule self-requested a review March 15, 2019 18:18
Copy link
Contributor

@zrphercule zrphercule left a comment

Choose a reason for hiding this comment

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

LGTM, great job!

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.

6 participants