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.

Sign up for GitHub

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

fix: Fixed load_pretrained_params in PyTorch when ignoring keys #902

Merged
merged 11 commits into from
Apr 28, 2022
Merged

fix: Fixed load_pretrained_params in PyTorch when ignoring keys #902

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
43814e3
fix: Fixed load_pretrained_params
Apr 27, 2022
f1f63a9
fix: Fixed ignore_keys for models in PyTorch
Apr 27, 2022
b51b217
test: Updated unittest accordingly
Apr 27, 2022
1bdedd2
refactor: Improved checks
Apr 27, 2022
2d28bf1
test: Added unittest for ignore_keys
Apr 27, 2022
b2a21dc
fix: Added missing import
Apr 28, 2022
79173f4
style: Fixed typing
Apr 28, 2022
de0652b
refactor: Refactored recognition ignore_keys
Apr 28, 2022
f8202ee
fix: Fixed typo
Apr 28, 2022
7de2ddc
fix: Fixed ResNet
Apr 28, 2022
5165d30
fix: Fixed load conditions
Apr 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refactor: Improved checks
  • Loading branch information
FG Fernandez committed Apr 27, 2022
commit 1bdedd2d34cd405a38ba77774157568b90e7ff7f
4 changes: 2 additions & 2 deletions doctr/models/utils/pytorch.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@ def load_pretrained_params(
for key in ignore_keys:
state_dict.pop(key)
missing_keys, unexpected_keys = model.load_state_dict(state_dict, strict=False)
if len(missing_keys) != len(ignore_keys) or len(unexpected_keys) > 0:
raise AssertionError("unable to load state_dict")
if set(missing_keys) != set(ignore_keys) or len(unexpected_keys) > 0:
raise ValueError("unable to load state_dict, due to non-matching keys.")
else:
# Load weights
model.load_state_dict(state_dict)
Expand Down