-
Notifications
You must be signed in to change notification settings - Fork 0
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
Flexibility of ratetables predictors #2
Comments
Oui je vois maintenant ... On verra pour lundi alors |
It will be the same issue for the test |
Yes, same thing for the test. |
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 ? |
So this should simply be closed. |
Supposons qu'a cet endroit il y a plusieurs covariablesd ans la table de mortalité (pas uniquement age, sex, year)
NetSurvival.jl/src/nonparamfit.jl
Lines 45 to 47 in de4ba47
Alors juste en dessous on passer
pred_g[:,1],pred_g[:,2]
etpred_g[:,3]
, et on oublie d'éventuelspred_g[:,4]
... Enfait il faudrait passer tout pred_g directement à la PoharPerme, a la place de age/year/sex uniquement. Mais ensuite lafunction _Λ(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.
The text was updated successfully, but these errors were encountered: