mlco2/codecarbon
Too many variables when initializing the EmissionsTracker
Aberta
#404 aberto em 5 de mai. de 2023
enhancementgood first issue
Métricas do repositório
- Stars
- (1.885 estrelas)
- Métricas de merge de PR
- (Métricas PR pendentes)
Description
Description
The BaseEmissionsTracker accepts 22 variables when initialized (and growing). This could lead to potential issues. And it is a known code smell.
What I Propose
Refactor the input variables to accept a config dict object as input.
So instead of
EmissionsTracker(project_name: "My project", measure_power_secs=10, output_dir="./temp")
codecarbon could do
config_options = {
project_name: "My project",
measure_power_secs=10,
output_dir="./temp",
}
EmissionsTracker(config_options)