compbiomed-unito / survhive

Convenient, opinionated wrapper around some (deep) survival models
MIT License
2 stars 0 forks source link

dataset flchain contains elements with zero time #30

Closed IvanUniTO closed 7 months ago

IvanUniTO commented 8 months ago

maybe this is why auton chokes...

print(my_data_df.dataframe.head())

dataset: flchain age sex sample.yr kappa lambda flc.grp creatinine mgus time \ 0 97.0 0.0 1997 5.70 4.860 10 1.7 0.0 85.0
1 92.0 0.0 2000 0.87 0.683 1 0.9 0.0 1281.0
2 94.0 0.0 1997 4.36 3.850 10 1.4 0.0 69.0
3 92.0 0.0 1996 2.42 2.220 9 1.0 0.0 115.0
4 93.0 0.0 1996 1.32 1.690 6 1.1 0.0 1039.0

event
0 1.0
1 1.0
2 1.0
3 1.0
4 1.0

X, y= my_data_df.get_X_y() numpy.argwhere(y['time'] == 0)

array([[ 28], [ 51], [691]])

IvanUniTO commented 8 months ago

after removal of the three zero-time elements, Auton's DSM now converges for the first time on flchain. 8:(

IvanUniTO commented 8 months ago

TODO: Add checks for zero-time in the test set. Possibly do this check even within the fit wrappers

IvanUniTO commented 7 months ago

added test functions within version 0.8