-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
perf: optimize Rcpp #411
perf: optimize Rcpp #411
Conversation
I see some |
Yeah thats for passing args by reference and hence avoiding the copy. I believe dev-check if failing due to mlr3 changes, but will check it out. |
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.
Max, for the c_weight_survival_score
I think you've just changed the "spacing" of the code, right (not the logic)? I can't check line-to-line here, so please verify! (I had checked that the C code worked even though it is very complicated as we use it in a paper)
Not quite, the args also passed by reference instead of copy. |
…to surv-assert
No description provided.