Automate default policies update process#18
Automate default policies update process#18jopemachine wants to merge 15 commits intolablup:mainfrom
Conversation
ff69af1 to
b4a733b
Compare
| return super(OYAMLDumper, self).increase_indent(flow, False) | ||
|
|
||
|
|
||
| def lense(obj, *args): |
There was a problem hiding this comment.
I wrote this utility function to simplify codes,
but I think utility like dot-prop looks also good.
get_property({'foo': {'bar': 'unicorn'}}, 'foo.bar');
//=> 'unicorn'
get_property({'foo': {'bar': 'a'}}, 'foo.notDefined.deep');
//=> NoneThere was a problem hiding this comment.
We actually already have one implementation 😉: https://github.com/lablup/backend.ai/blob/e2e18a2/src/ai/backend/common/utils.py#L107
There was a problem hiding this comment.
If there is a well-packaged 3rd-party library with a clean API, I think it is also good to use one.
There was a problem hiding this comment.
I copied and pasted the nmget code, but maybe making it as a separate lib named nmget might be better idea to maintain.
| return super(OYAMLDumper, self).increase_indent(flow, False) | ||
|
|
||
|
|
||
| def lense(obj, *args): |
There was a problem hiding this comment.
We actually already have one implementation 😉: https://github.com/lablup/backend.ai/blob/e2e18a2/src/ai/backend/common/utils.py#L107
| return super(OYAMLDumper, self).increase_indent(flow, False) | ||
|
|
||
|
|
||
| def lense(obj, *args): |
There was a problem hiding this comment.
If there is a well-packaged 3rd-party library with a clean API, I think it is also good to use one.
|
Ubuntu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
Uh oh!
There was an error while loading. Please reload this page.