Knowledge-Graph-Hub / neat-ml

Network Embedding All the Things
BSD 3-Clause "New" or "Revised" License
18 stars 1 forks source link

Update the update yaml script #47

Closed justaddcoffee closed 2 years ago

justaddcoffee commented 2 years ago
justaddcoffee commented 2 years ago

Making this a draft, because for some reason click can't find the update_yaml command

justaddcoffee commented 2 years ago

TODO:

justaddcoffee commented 2 years ago

Adding - @caufieldjh I've made a unit test here for this - unskip this once you've implemented the nested key thing above

caufieldjh commented 2 years ago

TODO:

  • per convo with @caufieldjh, fix bug when updating multiple nested keys - solution for now is to support specifying nested keys like this --keys graph_data:graph:node_path --values my/new/node/path

Done - keys now need to be specified like key1:key2:key3, but a list of different keys can still be provided, e.g. key1:key2:key3,key4:key5,key6

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

91.9% 91.9% Coverage
0.0% 0.0% Duplication