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

Replace hawktracer with tracing-chrome #3283

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

lu-zero
Copy link
Collaborator

@lu-zero lu-zero commented Nov 16, 2023

It is feature equivalent, we can easily extend the tracing abilities to use tracy and coz later.

Copy link

codecov bot commented Nov 16, 2023

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (524ae8a) 88.09% compared to head (f83aca7) 88.16%.
Report is 1 commits behind head on master.

Files Patch % Lines
src/api/lookahead.rs 50.00% 2 Missing ⚠️
src/bin/muxer/y4m.rs 0.00% 1 Missing ⚠️
src/encoder.rs 92.30% 1 Missing ⚠️
src/lrf.rs 80.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3283      +/-   ##
==========================================
+ Coverage   88.09%   88.16%   +0.07%     
==========================================
  Files          86       86              
  Lines       33806    28249    -5557     
==========================================
- Hits        29780    24906    -4874     
+ Misses       4026     3343     -683     

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

@lu-zero lu-zero merged commit 1a77439 into xiph:master Nov 16, 2023
@lu-zero lu-zero deleted the replace-hawktracer branch November 16, 2023 17:31
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.

None yet

2 participants