Skip to content

Estimate velocity - #1

Open
ipa-josh wants to merge 3 commits into
mathias-luedtke:estimate_velocityfrom
ipa-josh:estimate_velocity
Open

Estimate velocity#1
ipa-josh wants to merge 3 commits into
mathias-luedtke:estimate_velocityfrom
ipa-josh:estimate_velocity

Conversation

@ipa-josh

Copy link
Copy Markdown
  • added two fixes
  • visual review ok :)

@mathias-luedtke

Copy link
Copy Markdown
Owner

If overflow is zero, the behaviour is not undefined: the estimation is turned off and the velocity is just passed, this was intended.
Have you run the unit tests on your version? They should fail, otherwise I have to fix them.

@mathias-luedtke

Copy link
Copy Markdown
Owner

HACK
mathias-luedtke pushed a commit that referenced this pull request Jul 26, 2016
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.

2 participants