Skip to content

[ONNX]Fix export of full_like#40063

Closed
KsenijaS wants to merge 12 commits intopytorch:masterfrom
KsenijaS:fix_full_like
Closed

[ONNX]Fix export of full_like#40063
KsenijaS wants to merge 12 commits intopytorch:masterfrom
KsenijaS:fix_full_like

Conversation

@KsenijaS
Copy link
Copy Markdown
Contributor

Fix export of full_like when fill_value is of type torch._C.Value.

This PR fixes a bug when exporting GPT2DoubleHeadsModel huggingface/transformers#4950

@KsenijaS
Copy link
Copy Markdown
Contributor Author

@neginraoof @BowenBao please review

@dr-ci
Copy link
Copy Markdown

dr-ci Bot commented Jun 16, 2020

💊 CI failures summary and remediations

As of commit 3ec897e (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 on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 25 times.

@KsenijaS KsenijaS changed the title Fix export of full_like [ONNX]Fix export of full_like Jun 16, 2020
Copy link
Copy Markdown
Contributor

@neginraoof neginraoof left a comment

Choose a reason for hiding this comment

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

Thanks! Can you add a test for this use-case of full_like?

Comment thread torch/onnx/symbolic_opset9.py
@zou3519 zou3519 requested a review from neginraoof June 23, 2020 21:10
@zou3519 zou3519 added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Jun 23, 2020
Comment thread torch/onnx/symbolic_opset9.py
Comment thread test/onnx/expect/TestOperators.test_full_like.expect
@KsenijaS
Copy link
Copy Markdown
Contributor Author

KsenijaS commented Jul 1, 2020

@houseroad can you please review? Thanks

@KsenijaS
Copy link
Copy Markdown
Contributor Author

KsenijaS commented Jul 6, 2020

@houseroad can you please take a look? Thanks.

Copy link
Copy Markdown
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.

LG, thanks

ir_version: 6
producer_name: "pytorch"
producer_version: "XXX"
producer_version: "1.7"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this expect file still used?

Copy link
Copy Markdown
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.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@houseroad merged this pull request in b2cc8a2.

csarofeen pushed a commit to csarofeen/pytorch that referenced this pull request Jul 7, 2020
Summary:
Fix export of full_like when fill_value is of type torch._C.Value.

This PR fixes a bug when exporting GPT2DoubleHeadsModel huggingface/transformers#4950

Pull Request resolved: pytorch#40063

Reviewed By: hl475

Differential Revision: D22398353

Pulled By: houseroad

fbshipit-source-id: 6980a61211fe571c2e4a57716970f474851d811e
csarofeen added a commit to csarofeen/pytorch that referenced this pull request Aug 16, 2020
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Fix export of full_like when fill_value is of type torch._C.Value.

This PR fixes a bug when exporting GPT2DoubleHeadsModel huggingface/transformers#4950

Pull Request resolved: pytorch#40063

Reviewed By: hl475

Differential Revision: D22398353

Pulled By: houseroad

fbshipit-source-id: 6980a61211fe571c2e4a57716970f474851d811e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants