KEMBAR78
Added BigBirdPegasus onnx config by nandwalritik · Pull Request #17104 · huggingface/transformers · GitHub
Skip to content

Conversation

@nandwalritik
Copy link
Contributor

@nandwalritik nandwalritik commented May 5, 2022

What does this PR do?

Added BigBirdPegasus OnnxConfig to make this model available for conversion.
Sorry in my last pull request I forced push with wrong commit hash, which closed that PR. So I created a new one with same changes. Please consider.
@chainyo @lewtun

Before submitting

RUN_SLOW=1 pytest tests/onnx/test_onnx_v2.py -s -k "bigbird_pegasus"

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented May 5, 2022

The documentation is not available anymore as the PR was closed or merged.

@chainyo
Copy link
Contributor

chainyo commented May 6, 2022

Oh you closed the other one?

@nandwalritik
Copy link
Contributor Author

nandwalritik commented May 6, 2022

Oh you closed the other one?

While rebasing, I mistakenly forced pushed with wrong commit hash which reset the head of my branch to latest commit from main branch. All my previous commits were lost and pull request got closed automatically, I didn't had any other option other than creating a new pull request.

Copy link
Member

@lewtun lewtun left a comment

Choose a reason for hiding this comment

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

Thanks for re-opening a clean PR @nandwalritik ! This all looks good to me and was previously approved in #17078, so will merge once the CI passes

@lewtun lewtun merged commit 215e068 into huggingface:main May 6, 2022
elusenji pushed a commit to elusenji/transformers that referenced this pull request Jun 12, 2022
* Add onnx configuration for bigbird-pegasus

* Modify docs
CaoE added a commit to CaoE/transformers that referenced this pull request Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants