Skip to content
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

RSFs #72

Merged
merged 51 commits into from
Nov 25, 2024
Merged

RSFs #72

merged 51 commits into from
Nov 25, 2024

Conversation

RaphaelS1
Copy link
Collaborator

@RaphaelS1 RaphaelS1 commented Aug 3, 2024

Done for now @adibender . As with other chapters, this first draft focuses on structure and high-level technical content ready for review. As before I'll then fix your comments and we can send out for review.

Copy link

github-actions bot commented Aug 3, 2024

Book 📖

@RaphaelS1 RaphaelS1 changed the title RSFs [WIP] RSFs Aug 4, 2024
@RaphaelS1 RaphaelS1 requested a review from adibender August 4, 2024 18:09
@adibender
Copy link
Collaborator

Done for now @adibender . As with other chapters, this first draft focuses on structure and high-level technical content ready for review. As before I'll then fix your comments and we can send out for review.

Looks good in general. I think in the first part, it's partially very detailed, whereas in the survival part, it could have more illustration, as this is the "unique" part of the book. Other books/blogs, etc. give good intros to decision trees.

@RaphaelS1 RaphaelS1 requested a review from adibender November 4, 2024 14:56
@RaphaelS1 RaphaelS1 requested a review from adibender November 21, 2024 12:55
Copy link
Collaborator

@adibender adibender left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.
I made some edits and few minor comments.
I still think log-rank test might not be intuitive to many, so having an survival tree example with visualization for two potential splits would go a long way.
I'd suggest to merge this part after you had another pass and I will open a separate branch for the competing risks stuf.

This is overcome by recognising that the Kaplan-Meier estimation results in a piece-wise function that can be linearly interpolated between training data.
@fig-surv-ranfor-bootstrap demonstrates this process for three decision trees (panel a), where the survival probability is calculated at all possible time points (panels b-c), and the average is computed with linear interpolation added between time-points (panel d).

![Bootstrapping Kaplan-Meier estimators across three decision trees (red, blue, green). Panel a) shows the individual estimates, b) shows the time points to aggregate the trees over, c) is the predicted survival probability from each tree at the desired time points, and d) is the average survival probabilities connected by a step function.](Figures/forests/bootstrap.png){#fig-surv-ranfor-bootstrap fig-alt="Four panels with 't' on x-axis and 'S(t)' on y-axis. Panel a) shows three survival functions as piece-wise linearly decreasing step functions. Panel (b) shows the same but with vertical dotted lines added at all time-points. Panel (c) shows dots at each of the intersections between the vertical lines and the plotted decision trees. Panel (d) shows a single line with dots which are the average of the points in panel (c)."}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The overploting in c) might be confusing to some

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you mean overplotting?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

som of the blue dots hidden by green dots, no?

@RaphaelS1 RaphaelS1 merged commit 7ee1796 into main Nov 25, 2024
1 check passed
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.

2 participants