mlco2/codecarbon
Too many variables when initializing the EmissionsTracker
Aperta
#404 aperta il 5 mag 2023
enhancementgood first issue
Metriche repository
- Star
- (1885 stelle)
- Metriche merge PR
- (Metriche PR in attesa)
Descrizione
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)