-
Notifications
You must be signed in to change notification settings - Fork 119
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] Documentation for 1.0 #111
Comments
@mikemckiernan , please add all the tickets that you are working on for v1.0in the ticket here. If you need help adding these tickets please slack me |
PR NVIDIA-Merlin/models#193 is meant to encompass issue NVIDIA-Merlin/models#192 |
I've added some check boxes for other docs related work I think is in scope (some of which I'm on the hook for, and some of which we'll want to figure out how to handle.) |
@EvenOldridge @benfred , do we need to revise the readme for NVTabular ? |
@EvenOldridge , @benfred @karlhigley @mikemckiernan - in NVTabular readme - There are some un related items in the readme such as container build process for other repos. Should this be cleaned ? There is documentaiton regarding examples for other repos. Should this be moved ? |
Sounds like the HugeCTR work here is done, if I understood correctly from @mikemckiernan? |
1.0 release is complete. Let's create a new docs ticket. |
Merlin:
Core:
Models:
NVTabular:
Note: There are some un related items in the readme such as container build process for other repos. Should this be cleaned ? There is documentaiton regarding examples for other repos. Should this be moved ?
Systems:
Transformers4Rec:
HugeCTR
Note: Michael was able to resolve the issue. Will have SOK and HCTR under one Sphinx doc. Will have it for 22.04Individual repo doc updates
Expecting inputs from engineering team. Michael to share updates on how it affects his other works
Support matrix
Create One Support Matrix for Merlin
[ ]Support matrix - Combining tables to Training and Inference instead of 6 ( after 22.04)
Aha! Link: https://nvaiinfa.aha.io/features/MERLIN-822
The text was updated successfully, but these errors were encountered: