Skip to content

Bax integration for Badger visual extension - #442

Merged
MitchellAV merged 9 commits into
mainfrom
develop-bax-integration
Jul 31, 2026
Merged

Bax integration for Badger visual extension#442
MitchellAV merged 9 commits into
mainfrom
develop-bax-integration

Conversation

@MitchellAV

Copy link
Copy Markdown
Collaborator

Fixes various issues with Bax Generator so that is can be used within badger extension

  • class name should use Field class with frozen=True
  • changes various names to be more appropriate
  • added algorithm validation
    both _supported_algorithms as well as algorithms outside of Xopt can be used as long as they have a class_path that points to the module
  • various type hinting improvements (not fully complete)

@MitchellAV

Copy link
Copy Markdown
Collaborator Author

@dylanmkennedy Verify that each of these changes I needed to make still work with the bax algorithms separate repository

@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 99.15254% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
xopt/generators/bayesian/bax_generator.py 97.50% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@dylanmkennedy dylanmkennedy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These changes work with bax_algorithms. Mitchell and I have verified that these changes are both necessary and sufficient BAX/Badger integration.

@MitchellAV
MitchellAV merged commit 46cc86a into main Jul 31, 2026
13 checks passed
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