Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

codecarbon monitor prints a "Saved to" path that ignores output_dir

Cerrado Apto para principiantes
  • ¿Algo no está bien?
#1,322 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 2 días

@davidberenstein1957 ya está trabajando en esto.

Desde el 12/8/2026.

  • #1331 de @davidberenstein1957 — abierto
  • #1409 de @sohammishra864-wq — cerrado sin fusionar

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Tranquilo
Stack tecnológico
python
Área
cli

Línea de trabajo

Comienza en codecarbon/cli/monitor.py:119-125 y sigue la configuración de output_file y output_dir hasta codecarbon/output_methods/file.py:49-52. Reproduce el comando monitor con CODECARBON_OUTPUT_DIR establecido y verifica después que el informe muestre la ruta real del CSV y omita la ruta cuando la salida CSV esté deshabilitada.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

The post-run report printed by codecarbon monitor -- <cmd> resolves the emissions file path from the file name only, so whenever output_dir is not the current working directory the CLI tells the user to look in the wrong place.

Reproduction

cd /home/user
CODECARBON_OUTPUT_DIR=/var/log/codecarbon codecarbon monitor -- ./train.sh

Prints:

   Saved to: /home/user/emissions.csv

while the CSV was actually written to /var/log/codecarbon/emissions.csv. Same happens with output_dir set in .codecarbon.config.

Root cause

codecarbon/cli/monitor.py:119-125 uses tracker._conf["output_file"] and calls os.path.abspath() on it. But output_file is only the basename (default "emissions.csv", codecarbon/emissions_tracker.py:567); the directory lives in a separate key output_dir (emissions_tracker.py:566). The two are joined only inside FileOutput (codecarbon/output_methods/file.py:49). os.path.abspath therefore resolves the bare name against the CWD, which is correct only for the default output_dir=".".

FileOutput does log the correct absolute path at construction (output_methods/file.py:50-52), but the CLI defaults to log_level="error" (cli/main.py, forwarded at cli/monitor.py:54), so that INFO line is suppressed and the wrong path is the only thing the user sees.

Expected vs actual

  • Expected: the report shows the path the CSV was actually written to (and shows nothing when CSV output is disabled).
  • Actual: <cwd>/emissions.csv, unconditionally, even when no file output handler exists.

Display-only defect; measurement and file writing are unaffected.

Lenguaje dominante
Python
Estrellas
1.9k
Forks
341
Merge medio
3 d 51 min
PR fusionados (30 d)
16

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de mlco2/codecarbon

Todos los issues de mlco2/codecarbon

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.