Skip to content

support multi-threading in eval_performance#489

Merged
wxyucs merged 1 commit intomainfrom
eval_multithreading
Mar 25, 2025
Merged

support multi-threading in eval_performance#489
wxyucs merged 1 commit intomainfrom
eval_multithreading

Conversation

@wxyucs
Copy link
Collaborator

@wxyucs wxyucs commented Mar 7, 2025

related to: #273

@wxyucs wxyucs added the kind/improvement Code improvements (variable/function renaming, refactoring, etc. ) label Mar 7, 2025
@wxyucs wxyucs requested a review from LHT129 March 7, 2025 10:28
@wxyucs wxyucs self-assigned this Mar 7, 2025
@wxyucs wxyucs requested a review from jiaweizone as a code owner March 7, 2025 10:28
@mergify mergify bot added the module/tools label Mar 7, 2025
@wxyucs wxyucs force-pushed the eval_multithreading branch from f5bc8f6 to 15a1368 Compare March 20, 2025 12:49
@pull-request-size pull-request-size bot added size/M and removed size/S labels Mar 20, 2025
@codecov
Copy link

codecov bot commented Mar 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

@@            Coverage Diff             @@
##             main     #489      +/-   ##
==========================================
- Coverage   90.58%   90.53%   -0.05%     
==========================================
  Files         182      182              
  Lines       11095    11095              
==========================================
- Hits        10050    10045       -5     
- Misses       1045     1050       +5     
Flag Coverage Δ
cpp 90.53% <ø> (-0.05%) ⬇️

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

Components Coverage Δ
common 95.28% <ø> (ø)
datacell 92.89% <ø> (+0.04%) ⬆️
index 88.51% <ø> (-0.12%) ⬇️
simd 87.31% <ø> (ø)

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 a978f6e...c6a3d02. Read the comment docs.

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

@wxyucs wxyucs force-pushed the eval_multithreading branch from 15a1368 to ed22ee8 Compare March 21, 2025 08:34
@pull-request-size pull-request-size bot added size/L and removed size/M labels Mar 21, 2025
@wxyucs wxyucs force-pushed the eval_multithreading branch from ed22ee8 to 6c0a3eb Compare March 21, 2025 08:37
@wxyucs wxyucs changed the title use multi-threading in eval_performance support multi-threading in eval_performance Mar 21, 2025
@wxyucs wxyucs force-pushed the eval_multithreading branch 2 times, most recently from 41770b6 to fb7935e Compare March 24, 2025 04:18
@wxyucs wxyucs requested review from ShawnShawnYou and inabao March 24, 2025 05:56
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

@wxyucs wxyucs force-pushed the eval_multithreading branch 2 times, most recently from 7ca72ae to aea6445 Compare March 25, 2025 03:44
@wxyucs wxyucs force-pushed the eval_multithreading branch from aea6445 to e34fa8d Compare March 25, 2025 08:42
Signed-off-by: Xiangyu Wang <wxy407827@antgroup.com>
@wxyucs wxyucs force-pushed the eval_multithreading branch from e34fa8d to c6a3d02 Compare March 25, 2025 08:44
Copy link
Collaborator

@jiaweizone jiaweizone left a comment

Choose a reason for hiding this comment

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

LGTM

@wxyucs wxyucs merged commit f10d747 into main Mar 25, 2025
23 checks passed
@wxyucs wxyucs deleted the eval_multithreading branch March 25, 2025 11:44
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/tools size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants