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 model.py #45

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

shivarajkarki
Copy link

As per Issue #30 we get error when we use ResNet50 as feature extractor. When we register hook to final layer of bottleneck of ResNet50 then proper pool dims are generated. expected pool dims [512, 1024, 2048] where as earlier code would generate poll dims [128, 256, 512]. Now with this commit we will get proper pool dims.

As per Issue gudovskiy#30 we get error when we use ResNet50 as feature extractor. When is register hook to final layer of bottleneck then proper pool dims are generated.
@shivarajkarki
Copy link
Author

This commit will make code run smoothly when resnet50 is selected as feature extractor.

added braces
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.

1 participant