-
Notifications
You must be signed in to change notification settings - Fork 113
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
[RMP] Set up unit tests for Merlin example notebooks #205
Comments
Migrate the NVT example notebook unit tests to use Testbook @EvenOldridge @rnyak @mikemckiernan Radek and me talked yesterday, if we need to restructure/move the NVTabular examples. The NVTabular example contains components, which are not relevant anymore (training, deployment). I am not sure, if it make sense to migrate the unittests to testbook, now. I think we should define the new structure, first and add the unittests for the notebooks which will stay and/or add the new unittests in the new repo (where they will move) |
Now that the initial unit tests are in place we should come up with a process for adding new ones as examples are added to the libraries. @bschifferer can you work with me and @benfred to define this. |
All
Merlin Models
Merlin Systems
Merlin
loader
submodule to build TF async dataloader NVTabular#224The text was updated successfully, but these errors were encountered: