Oliver Borchert
Oliver Borchert
> but a +16,379 −17,112 PR is way too large to review. Fair enough. However, I think breaking up the PR in the same way as for the introduction of...
Thanks for your extensive summary @jameslamb! Unfortunately, I cannot comment on either of (1), (2) and (3) since I haven't used any of these libraries recently. > LightGBM model objects...
I'm in favor of removing as well ✅
@jameslamb the AppVeyor CI fails to solve the environment. However, I can't replicate this behavior locally (even for Windows) -- any idea how to debug this?
Thanks for the hint @jameslamb, `tornado` caused some issues :)
All right, let's get rid of this PR then 😅
(I'll fix the end-to-end CI to be run correctly from forks in a separate PR)
Could you elaborate a little more what you mean? Would you prefer to have a single foldable markdown section for created, deleted, ... resources?
Interesting thought! I am not entirely sure how to implement the "drill-down" in a single plan since you need to persist the plan somewhere. One option would be to add...