Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Active Learning Strategy in BoFire #356

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

jduerholt
Copy link
Contributor

@jduerholt jduerholt commented Feb 23, 2024

This PR is still work in progress and implements an ActiveLearningStrategy in BoFire as discussed in issue #331.

@jdridder: this is my current status:

  • the overall framework is set up, the ActiveLearningStrategy inherits from the BotorchStrategy and can be used in combination with AnyActiveLearningAcquisitionFunction, which is currently only qNegIntPosVar and can be only used with one output feature.
  • to make it actually working you have to implement the _get_acqfs method within the functional model of the ActiveLearningStrategy.
  • feel free to add more acqfs, if you want to use the ones from @hvarfner PR SCoreBO and Bayesian Active Learning acquisition functions pytorch/botorch#2163, you have to add a validator that they can be used only in combination with a SaasSingleTaskGPSurrogate surrogate.

I would recomment that you just create a branch based on this branch and then create a new PR. Thank you very much for your support.

@jduerholt jduerholt marked this pull request as draft February 23, 2024 13:26
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.

1 participant