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

Fix a bug on sqfp16 test #1477

Merged
merged 1 commit into from
Feb 10, 2024
Merged

Conversation

heemin32
Copy link
Collaborator

Description

Fix a bug in #1474 where invalid max/min value of fp16 is used

Issues Resolved

N/A

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Heemin Kim <[email protected]>
Copy link

codecov bot commented Feb 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (e2d5bc5) 85.05% compared to head (6e444b6) 85.02%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1477      +/-   ##
============================================
- Coverage     85.05%   85.02%   -0.04%     
  Complexity     1277     1277              
============================================
  Files           167      167              
  Lines          5207     5207              
  Branches        493      493              
============================================
- Hits           4429     4427       -2     
- Misses          570      572       +2     
  Partials        208      208              

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

@heemin32 heemin32 merged commit 06ccdbc into opensearch-project:main Feb 10, 2024
52 of 63 checks passed
@heemin32 heemin32 deleted the fix-test branch February 10, 2024 00:42
opensearch-trigger-bot bot pushed a commit that referenced this pull request Feb 10, 2024
Signed-off-by: Heemin Kim <[email protected]>
(cherry picked from commit 06ccdbc)
heemin32 added a commit that referenced this pull request Feb 10, 2024
Signed-off-by: Heemin Kim <[email protected]>
(cherry picked from commit 06ccdbc)

Co-authored-by: Heemin Kim <[email protected]>
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