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

Move variance of fixed features check to predictive strategy #441

Merged
merged 8 commits into from
Oct 8, 2024

Conversation

bertiqwerty
Copy link
Contributor

@bertiqwerty bertiqwerty commented Oct 2, 2024

I think it is okay to check for variance in fixed features only in predictive strategies. This helps to add data from only one task to the random strategy (or any strategy without surrogate) in case you have a multi-task domain. This might be useful in case you have a stepwise strategy with a random step to get initial random data from one of the tasks where you might not have historical data of.

@bertiqwerty bertiqwerty changed the title Move more-than-one-task check to predictive strategy Move variance of fixed features check to predictive strategy Oct 7, 2024
@bertiqwerty bertiqwerty requested a review from jduerholt October 7, 2024 13:16
Copy link
Contributor

@jduerholt jduerholt left a comment

Choose a reason for hiding this comment

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

Looks fine for me, only see the question below ;)

)
)

fixed_nontasks = (
Copy link
Contributor

Choose a reason for hiding this comment

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

Just one question: what is the purpose of a multitask optimization when your historical data is only for one task and you only want candidates in the future for the same task?

Copy link
Contributor Author

@bertiqwerty bertiqwerty Oct 8, 2024

Choose a reason for hiding this comment

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

Sorry. I wasn't clear in my formulation above. I have historical data of task 1 and I want to optimize task 2. The data I measure during optimization is also for task 2.

@bertiqwerty bertiqwerty merged commit 53782de into main Oct 8, 2024
10 checks passed
@jduerholt jduerholt deleted the predictive-multitask-check branch October 8, 2024 12:12
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