Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skalierung der Gesamtleistung falsch #1635

Closed
Gubi2023 opened this issue May 19, 2024 · 10 comments
Closed

Skalierung der Gesamtleistung falsch #1635

Gubi2023 opened this issue May 19, 2024 · 10 comments
Assignees
Labels
bug Something isn't working fixed dev fixed

Comments

@Gubi2023
Copy link

Gubi2023 commented May 19, 2024

bei den neueren Devs gibt es ein Problem mit der x-Achsen-Skalierung bei der Anzeige der Gesamtleistung. Der linksseitige Inhalt wird bei jedem Abfrageintervall gelöscht und die Anzeige nach rechts geschoben, sodass links eine Lücke entsteht und rechts der Inhalt aus dem Anzeigefenster geschoben wird. Bei mir wird dann pünktlich jede halbe Stunde die Skalierung neu kalibriert und das Spiel geht von vorne los. siehe Screenshot

Screenshot 2024-05-19 160935

Mein Abfrage-Intervall beträgt 15 sec.
Bei 5-sekündigem Abfrage-Intervall scheint es aber zu funktionieren

@strepwo
Copy link

strepwo commented May 20, 2024

hab was gefunden:
in history.html, Funktion calcScale(), wird bei der Berechnung von ts_pad anscheinend davon ausgegangen, dass man den oberen Plot an seiner Dauer < 1800 sec erkennen kann:
s.ts_pad = (s.ts_dur < 1800) ? 0 : s.ts_start % 1800
Aber wenn man als Abfrage-Intervall z.B. 15 sec eingestellt hat, ist ts_dur = refresh * value.length = 15 * 256 = 3840 > 1800..

Ich gucke mal weiter.. eigentlich geht es nur darum, dass man schöne Plots hat, wo alle Werte angezeigt werden; und dass die Zeiten stimmen (vor allem rechts, bei den neusten Werten).. eine Fallunterscheidung 'oben' vs 'unten' sollte nicht nötig sein

@strepwo
Copy link

strepwo commented May 20, 2024

hab vermutlich eine Lösung, siehe patch:
history_patch.txt
bisher: der ts_pad wurde so eingestellt, dass der Plot der Messwerte mit dem Plot des Grid zusammenpasst
neu: der Plot des Grid wird mit der korrekten Zeit gemacht; dadurch kann ts_pad entfallen, und dann gibte auch keine Probleme mehr damit. Kurzer Test sieht gut aus: alle Werte da, Zeiten stimmen. Bitte testen

@Gubi2023
Copy link
Author

ich hoffe, das kann jemand übernehmen, ich kanns nicht testen....

@lumapu
Copy link
Owner

lumapu commented May 20, 2024

ich teste und werde es übernehmen, wenn es passt. @strepwo echt cool, dass du dich da reingefuchst hast und den Code 'verstanden' hast. Mir gefällt, dass es jetzt weniger Code als davor ist - wäre ja cool, wenn damit das Problem gelöst wurde.

@lumapu
Copy link
Owner

lumapu commented May 20, 2024

mit 5s Intervall geht's weiterhin, daher empfinde ich es mal als gut und schicke es später ins Rennen, vielen Dank nochmal @strepwo

grafik

@strepwo
Copy link

strepwo commented May 20, 2024

ok super
'Code verstanden' - so halb, ist schon etwas kompliziert.. eher ein 'educated guess' :-)

Was man jetzt noch machen könnte/sollte: die Werte für den unteren Tages-Plot sind momentan einfach jeder x.te Messwert - besser wäre, diese zu mitteln (im C-code), das gibt die Gesamtleistung besser wieder. Oder anders gesagt: ein anti-aliasing einbauen.

@lumapu
Copy link
Owner

lumapu commented May 20, 2024

evtl. willst du diesen Wunsch einfach in #1199 formulieren, dann vergessen wir es auch nicht 😊

lumapu added a commit that referenced this issue May 20, 2024
* fix plot of history graph #1635
@lumapu lumapu self-assigned this May 20, 2024
@lumapu lumapu added bug Something isn't working fixed dev fixed labels May 20, 2024
@Loetnase
Copy link

Wäre es möglich diese Info, wie auf der Seite Daten, auch mit darzustellen?
grafik
Ich schreibe es auch mal in die #1199

@Gubi2023
Copy link
Author

mit 15 sec- Abfrage funktioniert es jetzt tadellos, gelegentlich teste ich noch andere Abfrageintervalls.

@Gubi2023
Copy link
Author

Gubi2023 commented May 21, 2024

Test mit 30 und 25 sec. Abfrage ebenfalls ok.
Vielen Dank für den schnellen Fix!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed dev fixed
Projects
None yet
Development

No branches or pull requests

4 participants