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
[Feature store] Handling multiple targets of same type/name/path #886
Conversation
mlrun/datastore/targets.py
Outdated
+ targets_by_kind_name | ||
) | ||
overriding_name_target_types = [ | ||
t[0] for t in no_name_target_types_count.items() if t[1] > 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
t[0] for t in no_name_target_types_count.items() if t[1] > 1 | |
target for target, target_count in no_name_target_types_count.items() if target_count > 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for line 107
mlrun/datastore/targets.py
Outdated
] | ||
+ targets_by_kind_name | ||
) | ||
overriding_name_target_types = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overriding_name_target_types = [ | |
target_types_requiring_name = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for line 109
overriding_path_target_types | ||
) | ||
) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're only verifying here the targets that gets the default name
What about verifying that the case where you received several targets, that has names, but more than one target has the same name ?
https://jira.iguazeng.com/browse/ML-387