Błąd statystyk

Rozwiązane
odrzucone, panel-v3-błędy
DBanaszewski
DBanaszewski α-tester v3

Nic nie muszę tłumaczyć, wszystko na zdjęciu (chodzi o RAM) Screenshot_20171219-141730|281x500

SystemZ
SystemZ Admin lvlup.pro

@DBanaszewski Mógłbyś bardziej wyjaśnić? Chodzi Ci o dodanie większego odstępu między wykresami CPU i RAM a Download i Upload?

@anon10657637 dał łatkę ale ona zmienia % na MB i GB i póki co nie widzę za bardzo powiązania

DBanaszewski
DBanaszewski α-tester v3

Chodzi o to, że wskaźnik RAM bierze dane ze wskaźnika CPU.

DoreK
DoreK α-tester v3

Dokładnie - poza tym nie widzę sensu istnienia wskaźników dla downloadu czy uploadu, bo te cały czas w teorii są nieaktywne

anon10657637
anon10657637

DBanaszewski:

Chodzi o to, że wskaźnik RAM bierze dane ze wskaźnika CPU.

We wolnej chwili poprawie.

DoreK:

Dokładnie - poza tym nie widzę sensu istnienia wskaźników dla downloadu czy uploadu, bo te cały czas w teorii są nieaktywne

To co zamiast tego proponujesz? Pisz, a pewnie to uwzględnie i zaimplementuje w aktualnym PR ;)

anon10657637
anon10657637

@DBanaszewski @DoreK @SystemZ gotowe: https://github.com/lvlup-pro/spawn-frontend-material/pull/186/commits/2c081cf4c770c7216a022fdd01b45e0c62172bbf

Od teraz wszystko powinno być ok ;)

SystemZ
SystemZ Admin lvlup.pro

DBanaszewski:

Chodzi o to, że wskaźnik RAM bierze dane ze wskaźnika CPU

Faktycznie 😰 😱 Tyle razy na to patrzyłem i nie zauważyłem. Dobrze że są inni co to widzą i zgłaszają :slight_smile:

PR od @anon10657637 zaakceptowany, fix będzie w RC5

DBanaszewski
DBanaszewski α-tester v3

(co do błędu statystyk): Screenshot_20180116-085452|281x500 Ten download i upload nie za duży? 😄

anon10657637
anon10657637

DBanaszewski:

Ten download i upload nie za duży?

Też to zauważyłem, tyle że na PC po chwili się zeruje. W następnym PR też postaram się naprawić.

SystemZ
SystemZ Admin lvlup.pro

DBanaszewski:

Ten download i upload nie za duży? 😄

Implementowałem to na szybko i niezbyt dokładnie. Po kilku sekundach działa już w przybliżeniu poprawnie, byłbym wdzięczny gdyby @anon10657637 udało się to poprawić.

anon10657637
anon10657637

https://github.com/lvlup-pro/spawn-frontend-material/pull/190

Stwierdziłem że w sumie ten wykres mógłby zostać pusty, bo i tak pokazuje tylko aktualną prędkość łącza. Mam nadzieję że odpowiada ;)

SystemZ
SystemZ Admin lvlup.pro

Skoro jest to wykres to czemu ma być pusty? Jeśli ktoś zużywa połowę łącza to dzięki temu będzie to wiedzieć. Jeśli przyjmę zaś ten patch to już się nie dowie 😦