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

Reduce default simulation run counts #2531

Merged

Conversation

ssoloff
Copy link
Member

@ssoloff ssoloff commented Oct 24, 2017

Per #2529.

@codecov-io
Copy link

Codecov Report

Merging #2531 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2531      +/-   ##
============================================
- Coverage     20.56%   20.56%   -0.01%     
+ Complexity     5861     5860       -1     
============================================
  Files           831      831              
  Lines         73623    73623              
  Branches      12400    12400              
============================================
- Hits          15144    15141       -3     
- Misses        56393    56394       +1     
- Partials       2086     2088       +2
Impacted Files Coverage Δ Complexity Δ
...games/strategy/triplea/settings/ClientSetting.java 90.27% <100%> (ø) 15 <0> (ø) ⬇️
...rategy/triplea/attachments/UnitTypeComparator.java 35.71% <0%> (-7.15%) 10% <0%> (-1%)
...tegy/triplea/oddsCalculator/ta/OddsCalculator.java 44.06% <0%> (-0.32%) 13% <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 916b900...fd759b5. Read the comment docs.

@RoiEXLab RoiEXLab merged commit 5c7aa7e into triplea-game:master Oct 24, 2017
@ssoloff ssoloff deleted the reduce-default-simulation-run-counts branch October 24, 2017 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants