Skip to content

API reference: Add a sanity test that ensures samples are runnable #3190

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
Closed
shmoradims opened this issue Apr 3, 2019 · 1 comment
Closed

API reference: Add a sanity test that ensures samples are runnable #3190

shmoradims opened this issue Apr 3, 2019 · 1 comment
Assignees
@codemzs
Labels
documentation Related to documentation of ML.NET test related to tests

Comments

@shmoradims
Copy link

This is a simpler version of #2954.

We need to make sure all the samples are runnable. Basically we'd need to look inside Microsoft.ML.Samples with reflection and run all static methods called Example().
@shmoradims shmoradims added documentation Related to documentation of ML.NET test related to tests labels Apr 3, 2019
@codemzs codemzs self-assigned this Apr 3, 2019
@shmoradims
Copy link
Author

Already fixed by @codemzs as a utility that runs all the samples. It's not part of the tests for build-time reasons.

Closing for now. @codemzs please reopen if there's still work needed for this.

@ghost ghost locked as resolved and limited conversation to collaborators Mar 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@codemzs codemzs

Labels
documentation Related to documentation of ML.NET test related to tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@codemzs @shmoradims