Update config.rst in "The YAML interface"#138
Update config.rst in "The YAML interface"#138SaidevPolisetty wants to merge 1 commit intoMatterMiners:masterfrom
Conversation
Added definition for high allocation and low utilisation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #138 +/- ##
=======================================
Coverage 92.68% 92.68%
=======================================
Files 33 33
Lines 1108 1108
Branches 206 206
=======================================
Hits 1027 1027
Misses 59 59
Partials 22 22 ☔ View full report in Codecov by Sentry. |
maxfischer2781
left a comment
There was a problem hiding this comment.
Sorry for taking so long, but I don't see how that content fits in this location.
This section is on the general config/pipeline structure. The !LinearController and !CpuPool are just examples here, so documenting their parameters is at best a tangent and unrelated to the intended scope. In addition, the wording used isn't appropriate for COBaLD: drones and draining is a concept exclusive to TARDIS.
If you can find a more suitable location and more generic wording, this could fit.
Added definition for high allocation and low utilisation