Skip to content

refactor some recall test on random dataset#520

Merged
LHT129 merged 1 commit intoantgroup:mainfrom
LHT129:recall
Mar 25, 2025
Merged

refactor some recall test on random dataset#520
LHT129 merged 1 commit intoantgroup:mainfrom
LHT129:recall

Conversation

@LHT129
Copy link
Collaborator

@LHT129 LHT129 commented Mar 21, 2025

No description provided.

@LHT129 LHT129 added kind/improvement Code improvements (variable/function renaming, refactoring, etc. ) version/0.14 labels Mar 21, 2025
@LHT129 LHT129 self-assigned this Mar 21, 2025
@pull-request-size pull-request-size bot added size/M and removed size/S labels Mar 21, 2025
Copy link
Collaborator

@inabao inabao left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@wxyucs wxyucs left a comment

Choose a reason for hiding this comment

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

lgtm

@LHT129 LHT129 force-pushed the recall branch 2 times, most recently from a477eda to ee8f194 Compare March 21, 2025 06:31
Signed-off-by: LHT129 <tianlan.lht@antgroup.com>
@codecov
Copy link

codecov bot commented Mar 21, 2025

Codecov Report

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

❌ Your patch status has failed because the patch coverage (75.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

@@            Coverage Diff             @@
##             main     #520      +/-   ##
==========================================
- Coverage   90.66%   90.60%   -0.06%     
==========================================
  Files         177      177              
  Lines       10846    10850       +4     
==========================================
- Hits         9833     9831       -2     
- Misses       1013     1019       +6     
Flag Coverage Δ
cpp 90.60% <75.00%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
common 95.24% <ø> (ø)
datacell 93.49% <100.00%> (+<0.01%) ⬆️
index 88.47% <50.00%> (-0.10%) ⬇️
simd 87.27% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a227a2...b16f9b3. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LHT129 LHT129 merged commit 3d3c3a8 into antgroup:main Mar 25, 2025
22 of 23 checks passed
@LHT129 LHT129 deleted the recall branch May 21, 2025 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/improvement Code improvements (variable/function renaming, refactoring, etc. ) module/testing size/M version/0.14

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants