Johannes P. Dürholt

Results 150 comments of Johannes P. Dürholt

Hi @dlinzner-bcs, could you explain a bit more detailed what the change is actually about and how it is implemented? Best, Johannes

> Has anyone used those so far? Is your question referring to the DoE module or to the whole package? Could be that it takes until end of the weekend...

@dlinzner-bcs: I try to review over the long weekend. Sorry for the delay!

@dlinzner-bcs: Tell me when I should review ;)

Fine, I will look into it until beginning of next week. Thanks for providing it!

> Hi Johannes. We need this pretty quickly. I would propose that @dlinzner-bcs fixes the minor things that you mentioned and we do the code de-duplication in a separate PR....

@CompRhys: I let some comments, when we have decided on how to handle these things, I would then have a look at the tests in the next iteration.

@CompRhys: short info: I try to get over it, today or tomorrow. Best, Johannes

Hi @CompRhys, I saw that you made some commits but also that you transferred it into a draft PR. Just ping me here, when I should review again! Best, Johannes

Ok, thanks for the info! I saw some nice stuff from you regarding SEBO in botorch/Ax. Nice! At some point, I also have to make it available here ;)