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

[TEST/CI] Add tests covering KerasModelArtifact #722

Closed
parano opened this issue May 27, 2020 · 1 comment · Fixed by #1295
Closed

[TEST/CI] Add tests covering KerasModelArtifact #722

parano opened this issue May 27, 2020 · 1 comment · Fixed by #1295
Assignees

Comments

@parano
Copy link
Member

parano commented May 27, 2020

Currently, there isn't any integration test covering the KerasModelArtifact:

@parano parano changed the title [TEST/CI] Add tests covering KerasModelArtifact #721 [TEST/CI] Add tests covering KerasModelArtifact May 27, 2020
@parano parano added good-first-issue Good for newcomers help-wanted An issue currently lacks a contributor test labels May 27, 2020
@parano parano added MLH and removed good-first-issue Good for newcomers labels Jul 13, 2020
@parano parano removed the MLH label Sep 13, 2020
@yubozhao yubozhao added the MLH label Sep 26, 2020
@parano
Copy link
Member Author

parano commented Nov 24, 2020

For @bojiang to review related tests, feel free to close if you think those tests are in good shape

@parano parano removed MLH help-wanted An issue currently lacks a contributor labels Nov 24, 2020
@bojiang bojiang linked a pull request Dec 4, 2020 that will close this issue
22 tasks
@bojiang bojiang closed this as completed Dec 4, 2020
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 a pull request may close this issue.

3 participants