DBanaszewski | 2017-12-19 13:20:40 UTC | #1
Nic nie muszę tłumaczyć, wszystko na zdjęciu (chodzi o RAM)
anon10657637 | 2018-01-12 20:28:05 UTC | #2
https://github.com/lvlup-pro/spawn-frontend-material/pull/186
SystemZ | 2018-01-14 12:11:21 UTC | #3
@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 | 2018-01-14 12:25:52 UTC | #4
Chodzi o to, że wskaźnik RAM bierze dane ze wskaźnika CPU.
DoreK | 2018-01-14 15:10:35 UTC | #5
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 | 2018-01-14 15:50:21 UTC | #6
[quote=”DBanaszewski, post:4, topic:4680, full:true”]
Chodzi o to, że wskaźnik RAM bierze dane ze wskaźnika CPU.
[/quote]
We wolnej chwili poprawie.
[quote=”DoreK, post:5, topic:4680, full:true”]
Dokładnie - poza tym nie widzę sensu istnienia wskaźników dla downloadu czy uploadu, bo te cały czas w teorii są nieaktywne
[/quote]
To co zamiast tego proponujesz? Pisz, a pewnie to uwzględnie i zaimplementuje w aktualnym PR ;)
anon10657637 | 2018-01-15 11:17:28 UTC | #7
@DBanaszewski @DoreK @SystemZ gotowe:
https://github.com/lvlup-pro/spawn-frontend-material/pull/186/commits/2c081cf4c770c7216a022fdd01b45e0c62172bbf
Od teraz wszystko powinno być ok ;)
SystemZ | 2018-01-16 03:59:37 UTC | #8
[quote=”DBanaszewski, post:4, topic:4680, full:true”]
Chodzi o to, że wskaźnik RAM bierze dane ze wskaźnika CPU
[/quote]
Faktycznie :coldsweat: :scream:
Tyle razy na to patrzyłem i nie zauważyłem.
Dobrze że są inni co to widzą i zgłaszają :slightsmile:
PR od @anon10657637 zaakceptowany, fix będzie w RC5
DBanaszewski | 2018-01-16 07:57:47 UTC | #9
(co do błędu statystyk):
Ten download i upload nie za duży? :smile:
anon10657637 | 2018-01-16 08:02:26 UTC | #10
[quote=”DBanaszewski, post:9, topic:4680”]
Ten download i upload nie za duży?
[/quote]
Też to zauważyłem, tyle że na PC po chwili się zeruje. W następnym PR też postaram się naprawić.
SystemZ | 2018-01-17 13:46:58 UTC | #11
[quote=”DBanaszewski, post:9, topic:4680”]
Ten download i upload nie za duży? :smile:
[/quote]
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 | 2018-01-17 15:16:03 UTC | #12
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 | 2018-01-17 18:49:29 UTC | #13
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 :frowning:
anon10657637 | 2018-01-20 20:46:23 UTC | #14
Działa!
https://github.com/lvlup-pro/spawn-frontend-material/pull/190/commits/8a6bd7405648f293564db3ffe1979a7a65d287c2
Aylin | 2019-10-30 17:21:27 UTC | #15