- Notifications
You must be signed in to change notification settings - Fork26.3k
Add logical_not operator.#23839
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
gchanan commentedAug 7, 2019
@xuhdev: are all your recent changes to this stack just merges/rebases? I need to know if landing a prior version is safe. also, can you stop merging/rebasing? I already checked that the failures weren't related to this diff, and we have some internal systems that will reject commits if the version we are trying to merge isn't up-to-date with the the PR. I'm not sure if the merges will go through now or I'll have to do redo it. I realize we have no automated communication on this state so this is totally invisible to you, sorry about that! |
xuhdev commentedAug 7, 2019
@gchanan Yes, I just rebased this morning, in case there's any conflict. Sorry for the confusion! |
gchanan commentedAug 7, 2019
ok, I think it makes the most sense to punt on getting this into v1.2 -- mainly because we can't land anything right now. |
gchanan commentedAug 7, 2019
Also, no need to apologize -- thanks so much for your contributions, they are really great. |
xuhdev commentedAug 7, 2019
@gchanan No problem! Thanks for your review! |
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Close#23836Pull Requestresolved:#23839Differential Revision: [D16678301](https://our.internmc.facebook.com/intern/diff/D16678301)
Summary:Pull Requestresolved:pytorch/pytorch#23839Close #23836Pull Requestresolved:pytorch/pytorch#23839Test Plan: Imported from OSSDifferential Revision: D16678301Pulled By: gchananfbshipit-source-id: 54e7b3f3b04c577e239b88493247e1c036266774
Uh oh!
There was an error while loading.Please reload this page.
Stack fromghstack:
Close#23836
Pull Requestresolved:#23839
Differential Revision:D16678301