Skip to content

Conversation

@mrpollo
Copy link
Member

@mrpollo mrpollo commented Feb 26, 2016

use the monotonic clock for timeouts all over the API

fixes #592

@hamishwillee
Copy link
Contributor

@peterbarker YOu want to check this one out?

requirements.txt Outdated
mock>=1.3.0
six>=1.9.0
dronekit-sitl>=3.0,<=3.99999
monotonic==0.6
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason this is strictly version 0.6?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no particular reason, I'll send a new commit for <1.0

@peterbarker
Copy link
Contributor

👍 LGTM with the exception of that version check on monotonic.

@mrpollo
Copy link
Member Author

mrpollo commented Feb 29, 2016

thanks @peterbarker, we'll get this merged since its blocking a Solo release

tcr3dr added a commit that referenced this pull request Feb 29, 2016
@tcr3dr tcr3dr merged commit 6da2160 into master Feb 29, 2016
@tcr3dr tcr3dr deleted the rroche/time-warp branch February 29, 2016 21:24
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.

time.time is not monotonic

5 participants