microsoft / MLOS

MLOS is a project to enable autotuning for systems.
https://microsoft.github.io/MLOS
MIT License
136 stars 64 forks source link

Better naming for `config` and `params` in VM and SaaS-related services #582

Open motus opened 11 months ago

motus commented 11 months ago

It is very confusing to have two dicts for config and params as arguments to many methods in, e.g., AzureVMService and AzureConfigService. We should be more explicit in naming and explain the difference between these arguments in the docstrings.

bpkroth commented 10 months ago

One instance of this mismatch: in remote_exec() we take config but in start_host() we take params.