Skip to content

Update noveou_train_netvlad_v3.py#8

Open
galsh17 wants to merge 1 commit into
mpkuse:masterfrom
galsh17:patch-1
Open

Update noveou_train_netvlad_v3.py#8
galsh17 wants to merge 1 commit into
mpkuse:masterfrom
galsh17:patch-1

Conversation

@galsh17

@galsh17 galsh17 commented Jun 29, 2020

Copy link
Copy Markdown

fix keras generator bug

fix keras generator bug
@mpkuse

mpkuse commented Jun 29, 2020

Copy link
Copy Markdown
Owner

what did you change exactly?

I see that you only changed a line?

@galsh17

galsh17 commented Jun 29, 2020

Copy link
Copy Markdown
Author

yes, changing this line fixes the bug.
you can see that keras sequence is expecting you to implement a method which is called on_epoch_end and not on_epoch_start
https://keras.io/api/utils/python_utils/#sequence-class

@mpkuse

mpkuse commented Jun 29, 2020

Copy link
Copy Markdown
Owner

Earlier they used to have the on_start method. Anyways. do you also mind putting a comment on this and indicate your keras version please?

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