-
-
Notifications
You must be signed in to change notification settings - Fork 560
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
[BUG] AalenAdditive
regressor predicts improper survival function
#1606
Comments
This is mentioned here as an artifact of the model. My intuition for that is because of the additive form of the hazard function, rather multiplicative and exponentiated like Cox. |
Hm, I wouldn't agree that this is a valid explanation, @bachnguyen-tomo. Any non-negative, integrable function with infinite integral is a valid hazard function - this can be seen from writing the survival function as (this is a well-known proposition that relates the survial function and the hazard function/distribution) So, no matter what the above equates to, as long as In consequence of this theorem, there might be a bug? |
But, I suppose this answers the more pragmatic question sufficiently, on whether this is something that people would expect to happen. Given the note in the documentation, it seems that this is expected (in the social sense) behaviour of the algorithm, and in that sense, we could close this issue. |
PS @bachnguyen-tomo, in case you have some input on what models that produce full distributions should do in this case, contribution here would be appreciated: sktime/skpro#249 |
@fkiraly The equation above assumes that the hazard function is non-negative though, which is the main drawback of the regressor, it doesn't guarantee non-negative hazard. First page. |
The
AalenAdditive
regressor predicts improper survival functions, i.e., functions that are not monotonous decreasing, or staying in the expected range [0,1]. Observed withlifelines 0.28.0
.To reproduce:
The text was updated successfully, but these errors were encountered: