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

feat: add cython monotonic_time_coarse implementation #38

Merged
merged 4 commits into from
Dec 1, 2023
Merged

feat: add cython monotonic_time_coarse implementation #38

merged 4 commits into from
Dec 1, 2023

Conversation

bdraco
Copy link
Member

@bdraco bdraco commented Dec 1, 2023

This is ~44% faster vs the python implementation

Copy link

codecov bot commented Dec 1, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (05ea718) 100.00% compared to head (71dc4c3) 100.00%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #38   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         6    +1     
  Lines          210       229   +19     
  Branches        30        34    +4     
=========================================
+ Hits           210       229   +19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jaymunro
Copy link

jaymunro commented Jan 4, 2024

I'm walking in the dark here. Trying to help out with the Bermuda tracker integration referenced above.

If the existing code imports as:

from homeassistant.util.dt import monotonic_time_coarse

What should this be changed to for the new location?

@bdraco
Copy link
Member Author

bdraco commented Jan 4, 2024

or from bluetooth_data_tools import monotonic_time_coarse

@bugtoo
Copy link

bugtoo commented Jan 4, 2024

or from bluetooth_data_tools import monotonic_time_coarse

This works nicely, thanks ❤️

@jaymunro
Copy link

jaymunro commented Jan 4, 2024

Working here too. I'll add a PR to the integration

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