-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Update DESCRIPTION #1321
Update DESCRIPTION #1321
Conversation
use survival 3.6-4 Signed-off-by: Joe Zhu <joe.zhu@roche.com>
Unit Tests Summary 1 files 84 suites 1m 12s ⏱️ Results for commit 9b4b80f. ♻️ This comment has been updated with latest results. |
Unit Test Performance Difference
Additional test case details
Results for commit 2153ec8 ♻️ This comment has been updated with latest results. |
Code Coverage Summary
Diff against main
Results for commit: 946abec Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Hi Joe, Sorry about this - since therneau/survival#240 has been fixed I thought we could remove the workarounds for the partial argument matching, but I forgot to check which validated version was available. I have reverted the removal of the partial argument matching code as well. |
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.
Good to go!
Hi @edelarua , thanks so much for the changes. Actually I think it was a good change, but maybe we did it too soon? I like your changes, and I think we should reinstate them at some stage. I have a proposal, as survival is a package is that is potentially used by other packages as well. I am going to submit the new version to the autovalidation, and see if it can pass the validation. once we have the new version, we can then safely implemtnat this. |
use survival 3.6-4
Pull Request
Fixes #nnn