Skip to content

REFACTOR: Refactor and aligned pinpair model#1883

Merged
svandenb-dev merged 11 commits intomainfrom
refactor_pinpair_model
Mar 8, 2026
Merged

REFACTOR: Refactor and aligned pinpair model#1883
svandenb-dev merged 11 commits intomainfrom
refactor_pinpair_model

Conversation

@maxcapodi78
Copy link
Collaborator

now PinPairModel and PinPair are aligned between the two codes

@github-actions github-actions bot added the testing Anything related to testing label Mar 6, 2026
@codecov-commenter
Copy link

codecov-commenter commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 68.84273% with 105 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.28%. Comparing base (5cbae21) to head (5260c8f).
⚠️ Report is 3 commits behind head on main.

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

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1883      +/-   ##
==========================================
+ Coverage   70.20%   70.28%   +0.07%     
==========================================
  Files         261      261              
  Lines       42416    42460      +44     
==========================================
+ Hits        29779    29843      +64     
+ Misses      12637    12617      -20     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

svandenb-dev
svandenb-dev previously approved these changes Mar 6, 2026
Copy link
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

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

LGTM

maxcapodi78 and others added 3 commits March 6, 2026 18:07
deprecation of edb port creations
test fix
deprecation of edb port creations
test fix
# Conflicts:
#	tests/system/test_edb.py
@github-actions github-actions bot added documentation Improvements or additions to documentation grpc-transition Changes that need to be included in gRPC transition labels Mar 7, 2026
@svandenb-dev svandenb-dev self-requested a review March 7, 2026 08:31
svandenb-dev
svandenb-dev previously approved these changes Mar 7, 2026
Copy link
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

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

LGTM

@svandenb-dev svandenb-dev self-requested a review March 8, 2026 12:31
Copy link
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

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

LGTM after fixing circular import in grpc

@svandenb-dev svandenb-dev merged commit 7d6b001 into main Mar 8, 2026
43 of 45 checks passed
@svandenb-dev svandenb-dev deleted the refactor_pinpair_model branch March 8, 2026 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation grpc-transition Changes that need to be included in gRPC transition testing Anything related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants