> On Oct. 22, 2019, 6:31 p.m., Sridhar K wrote: > > addons/models/4000-MachineLearning/4010-ml_model.json > > Lines 123 (patched) > > <https://reviews.apache.org/r/71619/diff/7/?file=2169829#file2169829line123> > > > > I am not the right person for this comment. But, this name looks > > confusing....is it a tag or image url?
Anand is very experienced in ML, and he said this name is commonly used for that purpose. It contains where to get the container image of a model > On Oct. 22, 2019, 6:31 p.m., Sridhar K wrote: > > addons/models/4000-MachineLearning/4010-ml_model.json > > Lines 269 (patched) > > <https://reviews.apache.org/r/71619/diff/7/?file=2169829#file2169829line269> > > > > I am wondering if we should have a first class entity called "User" > > defined in base model and you have ml_user extend from it. what do you say? what is parent for the entity "user"? - Na ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71619/#review218332 ----------------------------------------------------------- On Oct. 22, 2019, 6:17 p.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71619/ > ----------------------------------------------------------- > > (Updated Oct. 22, 2019, 6:17 p.m.) > > > Review request for atlas, Austin Nobis, Ashutosh Mestry, Karthik Manamcheri, > Sridhar K, Madhan Neethiraj, and Sarath Subramanian. > > > Bugs: atlas-3464 > https://issues.apache.org/jira/browse/atlas-3464 > > > Repository: atlas > > > Description > ------- > > Define entities used for Machine Learning Governance > > > Diffs > ----- > > addons/models/4000-MachineLearning/4010-ml_model.json PRE-CREATION > > > Diff: https://reviews.apache.org/r/71619/diff/7/ > > > Testing > ------- > > verified it is valid json file > > > Thanks, > > Na Li > >