Skip to content

Fix relative import in vlarge batch ariatinal strategy tests#2693

Merged
kayween merged 1 commit intocornellius-gp:mainfrom
Balandat:fix_test_import
Jan 4, 2026
Merged

Fix relative import in vlarge batch ariatinal strategy tests#2693
kayween merged 1 commit intocornellius-gp:mainfrom
Balandat:fix_test_import

Conversation

@Balandat
Copy link
Collaborator

@Balandat Balandat commented Jan 4, 2026

This breaks the test in some settings since test is not a proper package so the import fails. This uses a relative import instead.

@Balandat Balandat requested a review from kayween January 4, 2026 01:31
@Balandat
Copy link
Collaborator Author

Balandat commented Jan 4, 2026

Oops my bad misread this. Needs a different fix; will update.

@kayween kayween self-assigned this Jan 4, 2026
The current code breaks in some settings since `test` is not a propoer package - this uses relative imports instead, which fixes this issue.
@Balandat
Copy link
Collaborator Author

Balandat commented Jan 4, 2026

Actually fixed now!

@kayween
Copy link
Collaborator

kayween commented Jan 4, 2026

thanks!

@kayween kayween merged commit 048e38d into cornellius-gp:main Jan 4, 2026
7 checks passed
@Balandat Balandat deleted the fix_test_import branch January 4, 2026 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants