search

JuliaSurv / NetSurvival.jl

A pure-Julia take on standard net survival routines
https://juliasurv.github.io/NetSurvival.jl/
MIT License
10 stars 0 forks source link

Flexibility of ratetables predictors #2

Closed lrnv closed 4 months ago

lrnv commented 5 months ago

Supposons qu'a cet endroit il y a plusieurs covariablesd ans la table de mortalité (pas uniquement age, sex, year)

https://github.com/JuliaSurv/NetSurvival.jl/blob/de4ba476b993f038e0e75d7e0b00a1b1bfeacf90/src/nonparamfit.jl#L45-L47

Alors juste en dessous on passer pred_g[:,1],pred_g[:,2] et pred_g[:,3], et on oublie d'éventuels pred_g[:,4]... Enfait il faudrait passer tout pred_g directement à la PoharPerme, a la place de age/year/sex uniquement. Mais ensuite la

function _Λ(T, Δ, age, year, sex, ratetable, grid)

il faudrait prendre en compte un pred_g complet... puis en tenir compte lors de l'appel aux tables de mortalité.

Je pense qu'il vaut mieux attendre que #1 soit mergé pour s'occuper de ça, ce sera plus simple avec les nouvelles tables de mortalités.

rimhajal commented 5 months ago

Oui je vois maintenant ... On verra pour lundi alors

rimhajal commented 5 months ago

It will be the same issue for the test

lrnv commented 5 months ago

Yes, same thing for the test.

lrnv commented 5 months ago

If you take a look at the comments i added in https://github.com/JuliaSurv/NetSurvival.jl/pull/1/files it points directly at the issue, but it will not be very hard to solve after #1 :)

Btw, if you have a fit function working for the test, it should be in this repository right ?

lrnv commented 4 months ago

So this should simply be closed.