Repository navigation
CAT-Prüfstand: Frist von 300 auf 900 s, mit den gemessenen Zahlen - #455
Merged
Merged
Conversation
tst_cat_funkgeraet riss auf main die 300-s-Frist (Lauf 43597444850, macOS Apple Silicon 2/2, ***Timeout 300,09 s) und machte main rot. Es haengt nicht. Dieselbe Datei, derselbe Shard, derselbe Test #90: d6bad41 (vor #452) Passed 237,60 s 240f4dc (mit #452) Passed 254,49 s 3da9e7b (mit #453) ***Timeout 300,09 s Bei 254 s Grundlast bleiben 15 % Luft; das reisst eine Lastschwankung, ohne dass sich am Pruefstand etwas geaendert haette -- #453 fasst ihn gar nicht an. Die 300 waren also von Anfang an zu knapp bemessen, was erst auffiel, als sie rissen. Der Vorgaenger desselben Falls steht im Kommentar: am 08.11. liefen die 120 s ab. 900 s sind das Dreieinhalbfache der gemessenen Dauer, damit die naechsten Faelle nicht gleich wieder dagegen laufen. Die Zahlen stehen im Kommentar, damit beim naechsten Mal nicht geraten werden muss, und mit dem Hinweis, wo der Posten sitzt: die Warterunden gegen das rigctld-Doppel, nicht der Fensteraufbau. Alle 14 Compliance-Skripte gruen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
tst_cat_funkgeraetriss auf main die 300-s-Frist und machte main rot —Lauf 43597444850,
macOS Apple Silicon (2/2),
***Timeout 300.09 sec.Es hängt nicht, es war zu eng bemessen
Dieselbe Datei, derselbe Shard, jeweils Test #90:
d6bad415d(vor #452)240f4dc4d(mit #452, XIT)3da9e7b16(mit #453)***Timeout300,09 sBei 254 s Grundlast bleiben 15 % Luft. Das reißt bei jeder
Lastschwankung auf dem Läufer — und zwar ohne dass sich am Prüfstand
etwas geändert hätte: #453 fasst
tst_cat_funkgeraetgar nicht an, erändert nur
tests/tst_schutzrahmen.cpp.Mein erster Verdacht war ein Hänger, weil der Lauf mitten in
verbunden_geht_das_feld_auch_ohne_spalte_auf()abbrach. Die dreiMessungen widerlegen das: der Prüfstand lief die ganze Zeit nahe am
Limit, #452 legte 17 s drauf, und dieser Lauf war etwas langsamer als
der vorige.
Warum 900 und nicht 400
Das ist der zweite Riss derselben Frist — im Kommentar steht schon, dass
am 08.11. die damaligen 120 s abliefen. Ein knapp nachgezogenes Limit
reißt beim nächsten Prüffall wieder. 900 s sind das Dreieinhalbfache der
gemessenen Dauer.
Die drei Zahlen stehen jetzt im Kommentar, damit beim nächsten Mal nicht
geraten werden muss, zusammen mit dem Hinweis, wo der Posten sitzt: die
Warterunden gegen das rigctld-Doppel, nicht der Fensteraufbau. Wer die
Dauer wirklich senken will, muss dort ansetzen — das ist eine eigene
Arbeit, nicht diese.
Geprüft
Alle 14 Compliance-Skripte grün. Eine Zeile in
tests/CMakeLists.txtplus Kommentar; kein Code angefasst.
🤖 Generated with Claude Code