Closed ens-lg4 closed 4 years ago
Hi Leo, Most of it looks fine. I just have two questions:
1) may I ask you to give an example of "pass_matching_tags_to" usage? I think it's a useful functionality and I would like to understand it a bit better. From what I see, it's backwards compatible so it is just for my knowledge...
2) File module/env/module.py Line 1587. Are you sure that 'tags' is always in dd? I just don't remember ;) ... I usually do dd.get('tags',[]) just in case.
Thanks!
@gfursin , thanks for the review!
For example, in https://github.com/dividiti/ck-mlperf/blob/master/program/image-classification-tflite-loadgen/.cm/meta.json the weights
dependency (with sort order 30) will resolve first, and it will then take a tag that starts with "side." and add it to the list of tags of the images
dependency (with sort order 35). So if we already picked (by any means) a model that worked with 224*224 images, its tag side.224
would be passed to the image dataset and would ignore datasets with other image sizes.
We probably even want to know about the ones that don't have tags, to diagnose a potential problem early :)
Cool! Thanks for the clarification @ens-lg4 . Both answers makes total sense! I am merging the PR.
September:
September:
September:
September:
September: