mlco2/codecarbon
Too many variables when initializing the EmissionsTracker
オープン
#404 opened on 2023/05/05
enhancementgood first issue
Repository metrics
- Stars
- (1,885 個のスター)
- PR merge metrics
- (PR metrics pending)
説明
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)