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

restrict one dim requantize scale bias size #5888

Merged
merged 8 commits into from
Feb 5, 2025

Conversation

nihui
Copy link
Member

@nihui nihui commented Jan 23, 2025

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Jan 23, 2025

Codecov Report

Attention: Patch coverage is 99.25373% with 1 line in your changes missing coverage. Please review.

Project coverage is 91.77%. Comparing base (48e1260) to head (8ebedb7).

Files with missing lines Patch % Lines
src/layer/mips/requantize_mips.cpp 99.01% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #5888       +/-   ##
===========================================
- Coverage   94.88%   91.77%    -3.12%     
===========================================
  Files         824      293      -531     
  Lines      264735    43820   -220915     
===========================================
- Hits       251203    40217   -210986     
+ Misses      13532     3603     -9929     

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

@github-actions github-actions bot added the arm label Jan 25, 2025
@github-actions github-actions bot added the mips label Jan 25, 2025
@tencent-adm
Copy link
Member

tencent-adm commented Jan 25, 2025

CLA assistant check
All committers have signed the CLA.

@nihui nihui changed the title [WIP] restrict one dim requantize scale bias size restrict one dim requantize scale bias size Feb 5, 2025
@nihui nihui merged commit 956bccd into Tencent:master Feb 5, 2025
41 of 80 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants