-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[Fix] Fix example #1074
[Fix] Fix example #1074
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1074 +/- ##
=======================================
Coverage 67.99% 67.99%
=======================================
Files 159 159
Lines 10426 10426
Branches 1897 1897
=======================================
Hits 7089 7089
Misses 2968 2968
Partials 369 369
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@mzr1996 may take a look. |
Should we add a test example? Users cannot evaluate training results from the example now. |
We can add a test example in a new PR. |
Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Original example didn't work and it is too complicated for new users. In this PR, I write a simpler example directly run by users.
Modification
BC-breaking (Optional)
No BC-breaking
Use cases (Optional)
Run example
Checklist