SzFMV2020-Tavasz / AutomatedCar-A

Working repository for the subject "Szoftverfejlesztés multinacionális vállalatoknál" @OE-NIK 2020 Spring Group A
2 stars 1 forks source link

#188 Remove Acceleration Models + Implement Automatic Transmission Leveling #193

Closed aether-fox closed 4 years ago

aether-fox commented 4 years ago

Previous merge request (#192) was accidentally merged, this adds last touches and further developments. Associated ticket: is still #188

Changes

image

codecov[bot] commented 4 years ago

Codecov Report

Merging #193 into TeamA3 will increase coverage by 3.03%. The diff coverage is 53.57%.

Impacted file tree graph

@@             Coverage Diff              @@
##             TeamA3     #193      +/-   ##
============================================
+ Coverage     41.95%   44.99%   +3.03%     
- Complexity      279      312      +33     
============================================
  Files            58       57       -1     
  Lines          2019     2056      +37     
  Branches        173      179       +6     
============================================
+ Hits            847      925      +78     
+ Misses         1121     1063      -58     
- Partials         51       68      +17     
Flag Coverage Δ Complexity Δ
#unittests 44.99% <53.57%> (+3.03%) 312.00 <24.00> (+33.00)
Impacted Files Coverage Δ Complexity Δ
...a/hu/oe/nik/szfmv/automatedcar/math/MathUtils.java 87.50% <0.00%> (-12.50%) 4.00 <0.00> (ø)
...k/szfmv/automatedcar/powertrain/ITransmission.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...automatedcar/powertrain/Transmission_Previous.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
.../nik/szfmv/automatedcar/powertrain/PowerTrain.java 26.41% <16.66%> (-2.76%) 4.00 <1.00> (ø)
...mv/automatedcar/powertrain/SimpleTransmission.java 54.45% <50.94%> (+44.00%) 22.00 <10.00> (+20.00)
...atedcar/powertrain/TransmissionLevelSuggester.java 61.29% <61.29%> (ø) 7.00 <7.00> (?)
...v/automatedcar/powertrain/CarTransmissionMode.java 72.72% <92.30%> (+34.26%) 7.00 <6.00> (+6.00)
pintergreg commented 4 years ago

@SzFMV2020-Tavasz/team-a3 Ez merge-elhető, mivel ketten maradtatok, már egyetlen review is elég. Csak azért lettem ide hozzáadva, hogy meglehessen a kettő, vagy van valami konkrét kérdés?

Nektek is látnotok kell, hogy aktív a Merge PR gomb.

aether-fox commented 4 years ago

@SzFMV2020-Tavasz/team-a3 Ez merge-elhető, mivel ketten maradtatok, már egyetlen review is elég. Csak azért lettem ide hozzáadva, hogy meglehessen a kettő, vagy van valami konkrét kérdés?

Nektek is látnotok kell, hogy aktív a Merge PR gomb.

Leginkább azért adlak hozzá, hogy lásd mi történik. Ha csak a master-be történő request-ekre szeretnéd rárakatni magad velünk, nekem az is tökéletesen megfelel.