-
Notifications
You must be signed in to change notification settings - Fork 47
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
Add Active Learning Userguide #273
Conversation
a49ee34
to
bcbac1c
Compare
@AdrianSosic PR awaiting review for 2 weeks |
Sorry, too much on my plate these days. Will try to do it tomorrow 👍🏼 |
0760f4c
to
93a43fd
Compare
93a43fd
to
6c4a973
Compare
@AdrianSosic Would be great if you could review this to get our number of open PRs down ;) |
f07ca06
to
59007f4
Compare
@AdrianSosic almost 4 weeks now |
I know, I've already started this morning. But you keep having too many "high prio" things for me that require immediate fix 🙈 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @Scienfitz, thx for the guide, I finally managed to review it. I gave it another iteration, take what you like and drop what you don't like (or ask me if unsure). In particular, I've changed the paragraph containing the explanation on qNIPV, which was not quite correct.
4af5239
to
f5e606d
Compare
Co-authored-by: AdrianSosic <adrian.sosic@merckgroup.com>
a20c72c
to
d162514
Compare
Adds a small active learning userguide.
PosteriorStandardDeviation
acquisition function as its relevant in this case