Skip to content

Bump detection (no demo), and analysis.#250

Open
DivvyCr wants to merge 17 commits into
masterfrom
bumps
Open

Bump detection (no demo), and analysis.#250
DivvyCr wants to merge 17 commits into
masterfrom
bumps

Conversation

@DivvyCr

@DivvyCr DivvyCr commented Jul 31, 2020

Copy link
Copy Markdown
Contributor

Attempt 1 at finding all player contact in a game; then, determining who was the attacker/victim.

@DivvyCr

DivvyCr commented Jul 31, 2020

Copy link
Copy Markdown
Contributor Author

Noticed Issue #56 and wanted to try my hand at developing an actual feature, lol

@lgtm-com

lgtm-com Bot commented Jul 31, 2020

Copy link
Copy Markdown

This pull request introduces 3 alerts when merging baf33dc into 5d4385d - view on LGTM.com

new alerts:

  • 2 for Unused local variable
  • 1 for Wrong number of arguments in a call

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@lgtm-com

lgtm-com Bot commented Aug 2, 2020

Copy link
Copy Markdown

This pull request introduces 2 alerts when merging 3d989c3 into 07297fe - view on LGTM.com

new alerts:

  • 2 for Unused local variable

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@codecov

codecov Bot commented Aug 2, 2020

Copy link
Copy Markdown

Codecov Report

Merging #250 into master will increase coverage by 0.18%.
The diff coverage is 98.98%.

@@            Coverage Diff             @@
##           master     #250      +/-   ##
==========================================
+ Coverage   90.79%   90.97%   +0.18%     
==========================================
  Files          99       99              
  Lines        4441     4531      +90     
  Branches      746      760      +14     
==========================================
+ Hits         4032     4122      +90     
  Misses        297      297              
  Partials      112      112              

@DivvyCr DivvyCr changed the title Bump detection (no demo), and analysis. WIP Bump detection (no demo), and analysis. Aug 3, 2020
@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@lgtm-com

lgtm-com Bot commented Aug 3, 2020

Copy link
Copy Markdown

This pull request fixes 1 alert when merging 38d8ef9 into 07297fe - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@lgtm-com

lgtm-com Bot commented Aug 3, 2020

Copy link
Copy Markdown

This pull request fixes 1 alert when merging aa9bd9f into 07297fe - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@DivvyCr

DivvyCr commented Aug 3, 2020

Copy link
Copy Markdown
Contributor Author

The current version should be more or less deployable, but it needs testing (no clue how do make those).

@lgtm-com

lgtm-com Bot commented Aug 3, 2020

Copy link
Copy Markdown

This pull request fixes 1 alert when merging 6922200 into 07297fe - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

NOTE: Have to change demo test, because game_stats.bumps is now more populated.
@lgtm-com

lgtm-com Bot commented Aug 4, 2020

Copy link
Copy Markdown

This pull request fixes 1 alert when merging d847e75 into 07297fe - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@lgtm-com

lgtm-com Bot commented Aug 6, 2020

Copy link
Copy Markdown

This pull request fixes 1 alert when merging cf987f5 into 07297fe - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@DivvyCr

DivvyCr commented Aug 6, 2020

Copy link
Copy Markdown
Contributor Author

All tests fail... 3_BUMPS asserts 5==3, and 4_BUMPS asserts 2==4 (even though 1 of the bumps is a demo, so it should be 3==4)

@lgtm-com

lgtm-com Bot commented Aug 6, 2020

Copy link
Copy Markdown

This pull request fixes 1 alert when merging eadf10e into 07297fe - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@lgtm-com

lgtm-com Bot commented Aug 6, 2020

Copy link
Copy Markdown

This pull request fixes 1 alert when merging 719b9e8 into 07297fe - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@DivvyCr DivvyCr changed the title WIP Bump detection (no demo), and analysis. Bump detection (no demo), and analysis. Aug 6, 2020
@lgtm-com

lgtm-com Bot commented Aug 8, 2020

Copy link
Copy Markdown

This pull request fixes 1 alert when merging 7be3305 into 07297fe - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@lgtm-com

lgtm-com Bot commented Aug 8, 2020

Copy link
Copy Markdown

This pull request fixes 1 alert when merging 04044a1 into 07297fe - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

@lgtm-com

lgtm-com Bot commented Oct 15, 2020

Copy link
Copy Markdown

This pull request fixes 1 alert when merging 4305e0e into 3e66f17 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@AppVeyorBot

Copy link
Copy Markdown

1 similar comment
@AppVeyorBot

Copy link
Copy Markdown

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