Skip to content

fix: Corregido el cálculo del tiempo pra el comando /status#37

Open
misanram wants to merge 1 commit intopythoncanarias:masterfrom
misanram:miguel-viendo-a-ver
Open

fix: Corregido el cálculo del tiempo pra el comando /status#37
misanram wants to merge 1 commit intopythoncanarias:masterfrom
misanram:miguel-viendo-a-ver

Conversation

@misanram
Copy link
Copy Markdown

@misanram misanram commented Apr 2, 2026

La función util.since recibe dos parámetros:
dt:
"""datetime is the date to calculate the difference from
reference. If not used, take the value from the current
datetime."""
y reference:
"""datetime is the datetime used to get the difference
ir delta. If not defined, default value is since the definition
of the function, this is,since the moment the current run of the
program started.

En la actualidad se llama a la función desde (bot.py:53) con un único parámetro y con el valor de "self.started_at", que es un valor constante y que contiene el valor datetime del momento del arranque del bot.

El parámetro "reference" tiene como valor por defecto datetime.datetime.now() Ese valor se evalúa en el momento de importar la función, con lo que queda con un valor constante durante la ejecución del bot, solo unas milésimas de segundo mayor que "self.started_a"

Al llamar a la función util.since, el valor de los parámetros queda así:
dt = self.started_at
reference = unos milisegundos mayor que el anterior

Posteriormente la lógica de la función devuelve de forma correcta 0 seconds, ya que el delta calculado es mucho menor de 1 sec

Solución propuesta:

  1. Dado que el objetivo de la función es calcular el tiempo que lleva el bot en funcionamiento, dt siempre será el momento actual, con lo que propongo eliminarlo de la la llamada a la función y calcularlo en la función.
  2. Eliminar el valor por defecto de reference para que siempre use el valor que se le pasa a la función "self.started_at"
  3. Adaptar los test a la función modificada.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant