mlco2/codecarbon
Too many variables when initializing the EmissionsTracker
Ouverte
#404 ouverte le 5 mai 2023
enhancementgood first issue
Métriques du dépôt
- Stars
- (1 885 étoiles)
- Métriques de merge PR
- (Métriques PR en attente)
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)