Nothing Special   »   [go: up one dir, main page]
Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ONNX] Support lstm_cell symbolic (#61476) #62757

Closed
wants to merge 7 commits into from
Closed

[ONNX] Support lstm_cell symbolic (#61476) #62757

wants to merge 7 commits into from

Conversation

BowenBao
Copy link
Collaborator
@BowenBao BowenBao commented Aug 4, 2021 •
edited
Loading

Stack from ghstack:

Support lstm_cell symbolic

Co-authored-by: jiafatom jiafa@microsoft.com

Differential Revision: D30349061

@jiafatom @BowenBao
[ONNX] Support lstm_cell symbolic (#61476)
782e3b8 
Support lstm_cell symbolic

Co-authored-by: jiafatom <jiafa@microsoft.com>

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor
facebook-github-bot commented Aug 4, 2021 •
edited
Loading

🔗 Helpful links

💊 CI failures summary and remediations

As of commit 277ebcd (more details on the Dr. CI page):

  • 1/2 failures possibly* introduced in this PR
    • 1/1 non-scanned failure(s)
  • 1/2 broken upstream at merge base 60cadd0 on Aug 17 from 7:28am to 12:27pm

1 job timed out:

  • pytorch_linux_xenial_py3_clang7_asan_test2

🚧 1 fixed upstream failure:

These were probably caused by upstream breakages that were already fixed.

Please rebase on the viable/strict branch (expand for instructions)

If your commit is older than viable/strict, run these commands:

git fetch https://github.com/pytorch/pytorch viable/strict
git rebase FETCH_HEAD

ci.pytorch.org: 1 failed

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.

This was referenced Aug 4, 2021
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Merged
@BowenBao
Update on "[ONNX] Support lstm_cell symbolic (#61476)"
731de50 
Support lstm_cell symbolic

Co-authored-by: jiafatom <jiafa@microsoft.com>

[ghstack-poisoned]
BowenBao added 4 commits August 6, 2021 11:18
@BowenBao
Update on "[ONNX] Support lstm_cell symbolic (#61476)"
4381836 
Support lstm_cell symbolic

Co-authored-by: jiafatom <jiafa@microsoft.com>

[ghstack-poisoned]
@BowenBao
Update on "[ONNX] Support lstm_cell symbolic (#61476)"
f5450f6 
Support lstm_cell symbolic

Co-authored-by: jiafatom <jiafa@microsoft.com>

[ghstack-poisoned]
@BowenBao
Update on "[ONNX] Support lstm_cell symbolic (#61476)"
c27f6c8 
Support lstm_cell symbolic

Co-authored-by: jiafatom <jiafa@microsoft.com>

[ghstack-poisoned]
@BowenBao
Update on "[ONNX] Support lstm_cell symbolic (#61476)"
fc2f15f 
Support lstm_cell symbolic

Co-authored-by: jiafatom <jiafa@microsoft.com>

[ghstack-poisoned]
@BowenBao BowenBao mentioned this pull request Aug 13, 2021
Closed
@SplitInfinity
Copy link

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

@BowenBao
Update on "[ONNX] Support lstm_cell symbolic (#61476)"
277ebcd 
Support lstm_cell symbolic

Co-authored-by: jiafatom <jiafa@microsoft.com>

Differential Revision: [D30349061](https://our.internmc.facebook.com/intern/diff/D30349061)

[ghstack-poisoned]
@msaroufim
Copy link
Member

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

@facebook-github-bot
Copy link
Contributor

@msaroufim merged this pull request in 99b154b.

@facebook-github-bot facebook-github-bot deleted the gh/BowenBao/100/head branch August 22, 2021 14:22
@garymm garymm mentioned this pull request Oct 8, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@neginraoof neginraoof Awaiting requested review from neginraoof

@shubhambhokare1 shubhambhokare1 Awaiting requested review from shubhambhokare1

Assignees
No one assigned
Labels
cla signed Merged open source
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@BowenBao @facebook-github-bot @SplitInfinity @msaroufim @pytorchbot @jiafatom