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.

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

apply torch int div to layoutlmv2 #15457

Conversation

ManuelFay
Copy link
Contributor

@ManuelFay ManuelFay commented Feb 1, 2022

What does this PR do?

Linked to #14853 (now closed - bad rebase)

@LysandreJik

@HuggingFaceDocBuilder
Copy link

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@ManuelFay
Copy link
Contributor Author

REALM tests seem to be failing - doesn't seem like they should be running ?

Copy link
Member

@LysandreJik LysandreJik left a comment

Choose a reason for hiding this comment

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

This looks good to me! @NielsRogge can you give it a last look and merge if it looks good to you too?

Thanks for your contributions @ManuelFay!

@github-actions
Copy link

github-actions bot commented Mar 3, 2022

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

Please note that issues that do not follow the contributing guidelines are likely to be ignored.

Copy link
Contributor

@NielsRogge NielsRogge left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for adding this (sorry for late reply).

@NielsRogge
Copy link
Contributor

@ManuelFay can you rebase with master such that we can merge your PR?

@ManuelFay ManuelFay force-pushed the remove-deprecation-warning-int-div-layoutlmv2 branch from 108b8ac to 280880a Compare March 31, 2022 22:45
@ManuelFay
Copy link
Contributor Author

ManuelFay commented Mar 31, 2022

Done, and I changed the path to the custom divide function (compatible with older torch versions). @NielsRogge

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Mar 31, 2022

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

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

Please note that issues that do not follow the contributing guidelines are likely to be ignored.

@NielsRogge
Copy link
Contributor

NielsRogge commented Apr 25, 2022

Hi @ManuelFay, apologies, for some reason this is still not merged. Can you rebase your PR with the main branch such that we can merge it? Apologies again for asking you this a second time.

Thanks!

@ManuelFay ManuelFay force-pushed the remove-deprecation-warning-int-div-layoutlmv2 branch from e278470 to 3e7e744 Compare April 25, 2022 20:02
@ManuelFay
Copy link
Contributor Author

No problem @NielsRogge ! Here you go, rebased ;)

@NielsRogge NielsRogge merged commit fa32247 into huggingface:main Apr 26, 2022
@ManuelFay ManuelFay deleted the remove-deprecation-warning-int-div-layoutlmv2 branch April 26, 2022 15:47
elusenji pushed a commit to elusenji/transformers that referenced this pull request Jun 12, 2022
* apply torch int div

* black linting fixup

* update path to torch_int_div

* clarify imports
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.

5 participants