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 vgg19.py #17

Closed
wants to merge 2 commits into from
Closed

Update vgg19.py #17

wants to merge 2 commits into from

Conversation

jeffin07
Copy link

@jeffin07 jeffin07 commented Aug 28, 2020

I have changed the vgg19.py,legacy/vgg19small.py files. In the init method of Vgg19 class, I have added allow_pickle=True to np.load() function so that errors like in #7 Can be solved as everyone try to load the model with the given weights

@thatbrguy
Copy link
Owner

Hey @jeffin07, thanks for the effort! I'll do a quick review when I get time and then update on this.

@thatbrguy
Copy link
Owner

Hey, sorry for the late response. I've now mentioned this PR in the known issues section in the README file (link). I have decided not to merge this for now since I have not recommended an exact NumPy version to the users.

In the known issues section linked above, I have asked users who would like to use a higher version of NumPy (which may have this issue) to check this PR out to see how they can fix the issue manually. Thank you for your contribution!

@thatbrguy thatbrguy closed this Jun 27, 2021
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.

2 participants