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

update the requirements and fix the _expand_mask import issue #705

Open
wants to merge 198 commits into
base: main
Choose a base branch
from

Conversation

maulikmadhavi
Copy link

Junnan Li and others added 30 commits September 16, 2022 10:33
update instructions for Unified Feature Extraction Interface
update documentation for feature extraction
fix typos
When loading ALBEF feature extractor, it loads pre-trained checkpoints.
Yet the default behavior of load_from_pretrained will rename bert keys.
As a result, weights for ALBEF text encoders are not properly loaded.
This can be confirmed by checking missing keys after loading weights.
This commit fixes the issue by adding an option to not rename the keys.
Other ALBEF task-specific architectures should not be affected.
Copy link

Thanks for the contribution! Unfortunately we can't verify the commit author(s): Maulik Madhavi. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, sign the Salesforce Inc. Contributor License Agreement and this Pull Request will be revalidated.

Copy link

Thanks for the contribution! Unfortunately we can't verify the commit author(s): Junnan Li <j***@s***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, refresh the status of this Pull Request.

Copy link

Thanks for the contribution! Before we can merge this, we need @dxli94 @schhoi to sign the Salesforce Inc. Contributor License Agreement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.