-
Notifications
You must be signed in to change notification settings - Fork 7
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
cindex equal 0 #61
Comments
Hi @tcoroller , I don't think the edits proposed are correct because it would lead to a division by 0 ( torchsurv/src/torchsurv/metrics/cindex.py Line 590 in 57845ed
The se should not be 0. Do you want to give me a reproducible code so I can check the error you got? |
Sounds good, let me try to do a reproducible example |
Unless we can do something like: |
I mean intuitively the c-index standard error should never be 0. I'm curious to know what can can lead to this! |
File "/home/corolth1/torchsurv/src/torchsurv/metrics/cindex.py", line 568, in _p_value_noether
raise ValueError(
ValueError: The standard error of concordance index must be a positive value.
The text was updated successfully, but these errors were encountered: