-
Notifications
You must be signed in to change notification settings - Fork 4
Add 6 new best practices and minor fixes #8
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
base: source
Are you sure you want to change the base?
Conversation
The value of site.best_practices[0] is null because the first best practices is at site.best_practices[1].
|
Hi, Thanks for the PR. Here is some initial feedback regarding the practices:
Moreover, judging by the nature of the practices, I would say they fit better to the Coding/Deployment group of practices, rather than the training practices. The way we engineer the inference APIs is similar to implementing continuous integration or shadow deployment. I would call the practice something along the lines of "Design a flexible inferene API than can be used for experimentation.". Flexibility implies both composition and fast experimentation.
My suggestion is to try to first debate the large structural changes (e.g., if you agree some practices should be merge) and afterwards modify them one by one. |
Motivation
I identified six new potential best practices through conducting two case studies on a production ML pipeline while designing GreatAI. I think these are generalisable enough to be useful for other practitioners.
I created this PR following the advice of @jstvssr.
Changes
The added best practices are:
I also added some smaller fixes:
unique_id-s because two used to have the same (24)Thank you for taking a look at my PR!