Skip to content

Graph-Rendering im Terminal verbessern#34

Merged
fabianschmeltzer merged 1 commit into
mainfrom
codex/verbessern-der-darstellung
May 5, 2026
Merged

Graph-Rendering im Terminal verbessern#34
fabianschmeltzer merged 1 commit into
mainfrom
codex/verbessern-der-darstellung

Conversation

@fabianschmeltzer
Copy link
Copy Markdown
Owner

Motivation

  • Die Graph-Darstellung zeigte bei kurzen Historien und Idle-Phasen falsche Linien/Verbinder und drückte wichtige Cache-/Container-Informationen auf sehr großen Terminals zu weit nach unten.
  • Ziel ist, Nullwerte visuell auf der X‑Achse zu halten, spurious vertikale Verbinder zu vermeiden und die Graph-Höhe zu begrenzen, damit die UI aussagekräftiger wirkt.

Description

  • Extrahiert eine Hilfsfunktion _graph_points zur sauberen Berechnung von Punktkoordinaten und rechtsbündigen Ausrichtung kurzer Historien (_graph_points/draw_line_graph).
  • Verändert draw_line_graph so, dass Nullwerte nicht als Punkte auf der Skala dargestellt werden und keine vertikalen Connector-Balken gezeichnet werden; der neueste Nicht-Null-Wert wird gezielt mit markiert.
  • Fügt draw_x_axis hinzu und passt die Graph-Legende/Hilfetext an, um klarzustellen, dass Nullwerte auf der Achse verbleiben (kein Fake-Traffic), und ersetzt die alte Basislinie-Zeichnung.
  • Reduziert die maximale Graph-Höhe über graph_max_fraction/graph_max_h, damit Health-, Cache- und Container-Infos sichtbar bleiben; aktualisiert zugehörige Texte; und ergänzt Smoke-Tests in tests/test_smoke.py für Punkt-Layout und Spike-Verhalten.

Testing

  • python3 -m py_compile bcache-monitor wurde erfolgreich ausgeführt (Datei kompiliert ohne Syntaxfehler).
  • pytest -q wurde ausgeführt und alle Tests liefen erfolgreich (7 passed).

Codex Task

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@fabianschmeltzer fabianschmeltzer merged commit 085fb07 into main May 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant