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

Jump to bottom

Remove history from Keras model card #869

Closed
merveenoyan opened this issue May 11, 2022 · 3 comments
Closed

Remove history from Keras model card #869

merveenoyan opened this issue May 11, 2022 · 3 comments
Milestone
v0.8

Comments

@merveenoyan
Copy link
Contributor

This is a minor change, I would like to get rid of history in model card before next sprint (which I’m planning to have on first week of june) given we save it as a JSON now (see #861 ) , because for models with more epochs it will look bad on the card as it will log every epoch, and we already have tensorboard traces and history JSON, wanted to be transparent before I move on with the decision.
Pinging @nateraw @osanseviero
@merveenoyan merveenoyan mentioned this issue May 12, 2022
Closed
@merveenoyan
Copy link
Contributor Author

@LysandreJik I'd love to get this in next release :)

@LysandreJik
Copy link
Member

Sounds good, adding it to the milestone!

@LysandreJik LysandreJik added this to the v0.7 milestone May 16, 2022
@merveenoyan merveenoyan mentioned this issue May 17, 2022
Merged
@LysandreJik LysandreJik mentioned this issue Jun 10, 2022
Closed
@LysandreJik
Copy link
Member

Closing as fixed by #876

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.8
Development

No branches or pull requests
2 participants
@LysandreJik @merveenoyan