SzFMV2018-Osz / AutomatedCar-B

Working repository for the subject "Szoftverfejlesztés multinacionális vállalatoknál" @OE-NIK 2018 ősz B csoport
0 stars 0 forks source link

feature(rpm and speed meter): ready #79

Closed bartaroland closed 6 years ago

bartaroland commented 6 years ago

meter design ready, placed on dashboard, refreshes the value on change

codecov[bot] commented 6 years ago

Codecov Report

Merging #79 into TeamB3 will decrease coverage by 4.88%. The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff             @@
##             TeamB3     #79      +/-   ##
===========================================
- Coverage     45.39%   40.5%   -4.89%     
  Complexity       79      79              
===========================================
  Files            29      30       +1     
  Lines           456     511      +55     
  Branches         21      27       +6     
===========================================
  Hits            207     207              
- Misses          232     287      +55     
  Partials         17      17
Impacted Files Coverage Δ Complexity Δ
.../java/hu/oe/nik/szfmv/visualization/Dashboard.java 0% <0%> (ø) 0 <0> (ø) :arrow_down:
...szfmv/visualization/elements/CircleCalculator.java 0% <0%> (ø) 0 <0> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3651f5b...00b5a09. Read the comment docs.

pintergreg commented 6 years ago

A számjegyek kicsit összefolynak (akár font beállítás is lehet, múltkor is volt, hogy linuxon valamiért más volt), de nem néz ki rosszul.

Tesztelni mondjuk nem tudtam. Úgy értem ez a branch még nem reagál billentyűkre. Jó, autó sincs végül.

bartaroland commented 6 years ago

Az a baj, hogy túl nagyok és így hosszúak a számok stringnek, esetleg azzal tudok csalni, hogy minden 2. számot kiszedem, mert ahol sok lenne egymás mellett pl fent, tényleg egybelógnak 🙈

pintergreg commented 6 years ago

@bartaroland

bartaroland commented 6 years ago

Hát igen, 5 jobban elfér, mint 15, na majd javítom, csak kell még 1 approve, hogy ezt a verziót ideiglenesen bemossuk, mert pullolnom kéne a branchet, mivel közbe az indexeket megcsináltam, és jó lenne, ha a dashboard up2date lenne.