Skip to content

Improve import performance for inference#689

Draft
Yc-Chen wants to merge 2 commits intoroboflow:developfrom
Yc-Chen:improve-import-performance-for-inference
Draft

Improve import performance for inference#689
Yc-Chen wants to merge 2 commits intoroboflow:developfrom
Yc-Chen:improve-import-performance-for-inference

Conversation

@Yc-Chen
Copy link

@Yc-Chen Yc-Chen commented Feb 16, 2026

What does this PR do?

Related Issue(s):
#442, #503

Type of Change

  • Bug fix (non-breaking change that fixes an issue)

Testing

  • I have tested this change locally
  • I have added/updated tests for this change

Test details:

Added test for import performance.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code where necessary, particularly in hard-to-understand areas
  • My changes generate no new warnings or errors
  • I have updated the documentation accordingly (if applicable)

Additional Context

@CLAassistant
Copy link

CLAassistant commented Feb 16, 2026

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented Feb 16, 2026

Codecov Report

❌ Patch coverage is 47.29730% with 39 lines in your changes missing coverage. Please review.
✅ Project coverage is 31%. Comparing base (82ad5d1) to head (34fa6d7).
⚠️ Report is 21 commits behind head on develop.

❌ Your patch check has failed because the patch coverage (47%) is below the target coverage (95%). You can increase the patch coverage or adjust the target coverage.
❌ Your project check has failed because the head coverage (31%) is below the target coverage (95%). You can increase the head coverage or adjust the target coverage.

❗ There is a different number of reports uploaded between BASE (82ad5d1) and HEAD (34fa6d7). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (82ad5d1) HEAD (34fa6d7)
Linux 5 4
py3.12 2 1
gpu 1 0
Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #689     +/-   ##
========================================
- Coverage       60%    31%    -29%     
========================================
  Files           52     55      +3     
  Lines         6734   7058    +324     
========================================
- Hits          4041   2212   -1829     
- Misses        2693   4846   +2153     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Yc-Chen Yc-Chen requested a review from Matvezy as a code owner February 18, 2026 10:37
@Borda
Copy link
Member

Borda commented Feb 18, 2026

it seems there are to many unrelated chnages, could you pls rebase to develop 🐰

@Borda Borda marked this pull request as draft February 19, 2026 00:00
@Borda Borda added the enhancement New feature or request label Feb 19, 2026
@Yc-Chen Yc-Chen force-pushed the improve-import-performance-for-inference branch from bdfd4df to 34fa6d7 Compare February 19, 2026 10:02
@Borda Borda mentioned this pull request Feb 24, 2026
2 tasks
@Borda Borda force-pushed the develop branch 3 times, most recently from a6e6ca0 to 0485141 Compare March 13, 2026 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request has conflicts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants