Skip to content

Default Observe time window to 7 days - #470

Closed
Paulo Lacerda (placerda) wants to merge 1 commit into
developfrom
placerda-observe-default-7-days
Closed

Default Observe time window to 7 days#470
Paulo Lacerda (placerda) wants to merge 1 commit into
developfrom
placerda-observe-default-7-days

Conversation

@placerda

Copy link
Copy Markdown
Collaborator

Muda a janela de tempo padrão da tela Observe de 24 horas para 7 dias quando /observe é aberto sem start/end na URL.

Pontos alterados

Local Antes Depois
src/agentops/agent/observe/queries.pyDEFAULT_LOOKBACK_HOURS 24 24 * 7
src/agentops/agent/observe/ui.pyDEFAULT_RANGE_HOURS 24 24 * 7
src/agentops/agent/observe/ui.py — JS DEFAULT_RANGE_MS (dentro de _OBSERVE_SCRIPT) 24 * 60 * 60 * 1000 7 * 24 * 60 * 60 * 1000

Grep adicional encontrou três referências de documentação que também diziam 24 hours e foram atualizadas: o docstring de default_lookback_window em queries.py, o docstring de módulo e o de render_filter_bar em ui.py, e docs/observe.md.

Sobre as duas constantes

DEFAULT_RANGE_HOURS (ui) e DEFAULT_LOOKBACK_HOURS (queries) foram mantidas duplicadas de propósito:

  • ui.py é um módulo de renderização puro — sem I/O de rede, sem env vars, sem Azure SDK, importando apenas stdlib. Importar queries.py traria agentops.core.attribution, agentops.core.cost e agentops.core.observe junto.
  • _OBSERVE_SCRIPT é uma string tripla comum (não f-string), então o valor JS não pode ser interpolado sem escapar todas as chaves do JavaScript.
  • O arquivo já segue a convenção "hardcode + comentário mirrors X" para MAX_TREND_POINTS e AUTO_REFRESH_MS.

Foi adicionado um comentário explícito em ui.py apontando o espelhamento, e os testes cobrem os dois valores.

Testes

  • tests/unit/test_observe_queries.py: test_default_lookback_window_is_24_hourstest_default_lookback_window_is_7_days, com assert adicional end - start == timedelta(days=7)
  • tests/unit/test_observe_ui.py: assert ui.DEFAULT_RANGE_HOURS == 24 * 7 e test_script_computes_default_24_hour_range_when_missing_from_url..._7_day_... verificando o novo literal JS

python -m pytest tests/unit/test_observe_queries.py tests/unit/test_observe_ui.py260 passed (após rebase em develop).

Nota: src/agentops/agent/time_range.py não foi tocado — é do cockpit e já usa 7 dias.

Change the Observe screen's default lookback from 24 hours to 7 days when no start/end is present in the URL. Updates the backend lookback constant, the UI range constant, and the mirrored constant in the embedded JavaScript, plus docstrings and docs.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@placerda
Paulo Lacerda (placerda) deleted the placerda-observe-default-7-days branch August 31, 2026 13:06
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