Skip to content

fix(metalearner_discrete): fixing the metalearner #157

Open
benesom wants to merge 2 commits intodevfrom
feat/improve-quadprog-in-metalearner_nnls
Open

fix(metalearner_discrete): fixing the metalearner #157
benesom wants to merge 2 commits intodevfrom
feat/improve-quadprog-in-metalearner_nnls

Conversation

@benesom
Copy link
Collaborator

@benesom benesom commented Mar 20, 2026

  • adding unit tests
  • ensuring now that Dmat argument to quadprog::solve.QP is always positive definite (setting eigenvalue to be at least sqrt(.Machine$double.eps))

@benesom benesom requested a review from kkholst March 20, 2026 09:04
@benesom
Copy link
Collaborator Author

benesom commented Mar 20, 2026

@kkholst, I created #158 to flag the missing documentation for the metalearners.

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