Skip to content

Adding Conv1d to quantization default_mappings#36352

Closed
z-a-f wants to merge 1 commit intogh/z-a-f/9/basefrom
gh/z-a-f/9/head
Closed

Adding Conv1d to quantization default_mappings#36352
z-a-f wants to merge 1 commit intogh/z-a-f/9/basefrom
gh/z-a-f/9/head

Conversation

@z-a-f
Copy link
Copy Markdown

@z-a-f z-a-f commented Apr 9, 2020

Stack from ghstack:

Differential Revision: D20955781

z-a-f pushed a commit that referenced this pull request Apr 9, 2020
ghstack-source-id: 64564db
Pull Request resolved: #36352
@z-a-f z-a-f requested a review from jerryzh168 April 9, 2020 23:57
@dr-ci
Copy link
Copy Markdown

dr-ci Bot commented Apr 9, 2020

💊 CircleCI build failures summary and remediations

As of commit 99baf1c (more details on the Dr. CI page):


  • 1/1 failures introduced in this PR

XLA failure

Job pytorch_xla_linux_xenial_py3_6_clang7_test is failing. Please create an issue with title prefixed by [PT_BREAK] in pytorch/xla and link to to this PR. If you have questions, please reach out to @ailzhang / @dlibenzi / @JackCaoG.


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.

See how this bot performed.

This comment has been revised 2 times.

nn.Linear: nnq.Linear,
nn.ReLU: nnq.ReLU,
nn.ReLU6: nnq.ReLU6,
nn.Conv1d: nnq.Conv1d,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need to change for qat mapping as well?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

We don't have qat conv1d

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I will send qat conv1d later (P1 for now :))

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@z-a-f merged this pull request in 891a533.

1 similar comment
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@z-a-f merged this pull request in 891a533.

@facebook-github-bot facebook-github-bot deleted the gh/z-a-f/9/head branch April 13, 2020 14:16
ashishfarmer pushed a commit to ashishfarmer/pytorch that referenced this pull request Apr 13, 2020
Summary: Pull Request resolved: pytorch#36352

Test Plan: Imported from OSS

Differential Revision: D20955781

Pulled By: z-a-f

fbshipit-source-id: 37fbcf329a6abcd9a367a73ad65ce543ed9ffe47
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary: Pull Request resolved: pytorch#36352

Test Plan: Imported from OSS

Differential Revision: D20955781

Pulled By: z-a-f

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants