KEMBAR78
Make torch.utils.bencmark numpy free by malfet · Pull Request #60564 · pytorch/pytorch · GitHub
Skip to content

Conversation

@malfet
Copy link
Contributor

@malfet malfet commented Jun 23, 2021

PyTorch core do not depend on numpy, so benchmarks should not depend on it as well

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Jun 23, 2021

💊 CI failures summary and remediations

As of commit b44d1ea (more details on the Dr. CI page and at hud.pytorch.org/pr/60564):


  • 1/1 failures introduced in this PR

1 failure not recognized by patterns:

Job Step Action
GitHub Actions Lint / shellcheck Assert that regenerating the workflows didn't change them 🔁 rerun

Preview docs built from this PR

This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@malfet malfet requested a review from ngimel June 23, 2021 18:41
@malfet malfet force-pushed the malfet/make-torch.utils.benchmarks-numpy-free branch from 1ef6e73 to 16c8809 Compare June 24, 2021 01:00
malfet added 4 commits June 30, 2021 09:38
torch.median() for tensors with even number of elements returns lower of two medians, while numpy returns average, which corresponds to `quantile(0.5)`
@malfet malfet force-pushed the malfet/make-torch.utils.benchmarks-numpy-free branch from 16c8809 to b44d1ea Compare June 30, 2021 17:44
@facebook-github-bot
Copy link
Contributor

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

@malfet
Copy link
Contributor Author

malfet commented Jun 30, 2021

I think it's ready for review now.

Copy link

@robieta robieta left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@facebook-github-bot
Copy link
Contributor

@malfet merged this pull request in cef0851.

@malfet malfet deleted the malfet/make-torch.utils.benchmarks-numpy-free branch June 30, 2021 21:48
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