-
Notifications
You must be signed in to change notification settings - Fork 21
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
Finished slide deck for w06 HPO BO. #8
base: master
Are you sure you want to change the base?
Conversation
@frank-hutter @KEggensperger Could you please comment on the status of this PR. It is very old and I don't know whether we should merge it or just drop it. |
@KEggensperger could you please comment on the status of this PR |
My best guess would be that we can drop the tex changes since the master branch contains more recent commits by Frank and I assume the master branch reflects what was used for the course. However, I would keep the plotting scripts, maybe as a separate file. |
I also remember @frank-hutter having merged the slides. I don't know if he merged the scripts, though. |
Hi everyone,
I fixed the slides based on the comments in there, but during the course
last year I never knew about the pull request. I have not done anything
with the scripts.
Cheers,
Frank
…On Tue 20. Apr 2021 at 20:23, Matthias Feurer ***@***.***> wrote:
I also remember @frank-hutter <https://github.com/frank-hutter> having
merged the slides. I don't know if he merged the scripts, though.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#8 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABLOK764COQTDZVCRQMGK6TTJXBB3ANCNFSM4M4QOQMA>
.
|
So, how to continue with the PR then? |
Barring any minor corrections that were unfortunately missed, the slide deck for week 6 - HPO BO should be considered complete at this point.