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

docs (model): corrected default for tf_epochs #221

Closed
wants to merge 4 commits into from
Closed

docs (model): corrected default for tf_epochs #221

wants to merge 4 commits into from

Conversation

ottkat
Copy link

@ottkat ottkat commented Jun 12, 2024

corrected default value of tf_epochs in the documentation to match the code

@muellerdo
Copy link
Member

Nice! :)

Could you add your unit testing for the DataGenerator metadata, as well?

@muellerdo muellerdo self-requested a review June 12, 2024 13:36
@muellerdo muellerdo self-assigned this Jun 12, 2024
@muellerdo
Copy link
Member

The Setup testing could be faulty, right now, as we are still waiting for the vision transformer subpackage repo to check/accept my PR to support Keras 3.

@muellerdo
Copy link
Member

No space in commits between type and (scope)
e.g. docs(model) instead of docs (model)

@ottkat ottkat closed this by deleting the head repository Jun 18, 2024
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