Skip to content

Comments

Correct Active Search#1

Open
WardLT wants to merge 40 commits intotheodore-ando:masterfrom
WardLT:master
Open

Correct Active Search#1
WardLT wants to merge 40 commits intotheodore-ando:masterfrom
WardLT:master

Conversation

@WardLT
Copy link

@WardLT WardLT commented Oct 5, 2018

From my understanding, the current implementation of active search is missing the probability of the choice point in the effective utility. This PR changes the scoring function for active search to use the probability of the point being the positive class, which recovers the behavior such that if the budget is 1 then the active search is equivalent to greedy selection.

@WardLT
Copy link
Author

WardLT commented Oct 5, 2018

It also adds another example notebook

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