mlco2/codecarbon

Too many variables when initializing the EmissionsTracker

开放

#404 创建于 2023年5月5日

 (3 条评论) (0 个反应) (0 位负责人)Python (306 个派生)auto 404
enhancementgood first issue

仓库指标

星标
 (1,885 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

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)

贡献者指南