Skip to content

Conversation

soulitzer
Copy link
Contributor

@soulitzer soulitzer commented Dec 2, 2020

@dr-ci
Copy link

dr-ci bot commented Dec 2, 2020

💊 CI failures summary and remediations

As of commit 50cd680 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


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.

This comment has been revised 97 times.

@soulitzer soulitzer changed the title Implements np.sinc Implement np.sinc Dec 3, 2020
@soulitzer soulitzer changed the title Implement np.sinc Add sinc operator Dec 3, 2020
@codecov
Copy link

codecov bot commented Dec 4, 2020

Codecov Report

Merging #48740 (50cd680) into master (eb131cf) will decrease coverage by 0.00%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##           master   #48740      +/-   ##
==========================================
- Coverage   80.61%   80.61%   -0.01%     
==========================================
  Files        1879     1879              
  Lines      203539   203549      +10     
==========================================
+ Hits       164074   164081       +7     
- Misses      39465    39468       +3     

@soulitzer soulitzer marked this pull request as ready for review December 11, 2020 00:18
@soulitzer soulitzer requested a review from mruberry December 11, 2020 00:18
Copy link
Collaborator

@mruberry mruberry left a comment

Choose a reason for hiding this comment

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

Nice work, @soulitzer! Just some minor nits for cleanup before merging. Thanks for being so diligent with adding an OpInfo for sinc.

@mruberry
Copy link
Collaborator

The asan_test2 build failure is unrelated to this PR.

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.

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

@facebook-github-bot
Copy link
Contributor

@soulitzer merged this pull request in bbc7143.

@facebook-github-bot
Copy link
Contributor

This pull request has been reverted by 7767dcf.

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.

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

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.

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

@mruberry mruberry mentioned this pull request Dec 20, 2020
hwangdeyu pushed a commit to hwangdeyu/pytorch that referenced this pull request Jan 6, 2021
Summary:
Implements the sinc operator.
See https://numpy.org/doc/stable/reference/generated/numpy.sinc.html

![image](https://user-images.githubusercontent.com/13428986/101653855-cdffa080-3a0d-11eb-8426-ecc81c152ebd.png)

Pull Request resolved: pytorch#48740

Reviewed By: izdeby

Differential Revision: D25564477

Pulled By: soulitzer

fbshipit-source-id: 13f36a2b84dadfb4fd1442a2a40a3a3246cbaecb
hwangdeyu pushed a commit to hwangdeyu/pytorch that referenced this pull request Jan 6, 2021
Summary:
Implements the sinc operator.
See https://numpy.org/doc/stable/reference/generated/numpy.sinc.html

![image](https://user-images.githubusercontent.com/13428986/101653855-cdffa080-3a0d-11eb-8426-ecc81c152ebd.png)

Pull Request resolved: pytorch#48740

Reviewed By: ezyang

Differential Revision: D25597565

Pulled By: soulitzer

fbshipit-source-id: 6dbcf282ee4eba34930bc9e5c85c0c5e79cf0322
@soulitzer soulitzer deleted the implement-sinc branch April 14, 2021 21:31
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