-
Notifications
You must be signed in to change notification settings - Fork 4
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
Fix generating quantities not using correct max value #306
Conversation
actually consider this as draft just noticed some small things I need to clean up |
Unit Tests Summary 1 files 41 suites 7m 36s ⏱️ Results for commit 3ae4c58. ♻️ This comment has been updated with latest results. |
Unit Test Performance Difference
Additional test case details
Results for commit 6f3f7d0 ♻️ This comment has been updated with latest results. |
Ok i think this should be ready now |
Code Coverage Summary
Diff against main
Results for commit: 3ae4c58 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Closes #305
Issue was code incorrectly always looked for last tumour time rather than last event time when generating survival quantities