Skip to content
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.

Sign up for GitHub

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

Jump to bottom

'cosine' kernel in FastKernelSurvivalSVM still in documentation but not working in 0.22.2 #439

Closed
aliciaolivaresgil opened this issue Mar 19, 2024 · 0 comments · Fixed by #444
Closed

'cosine' kernel in FastKernelSurvivalSVM still in documentation but not working in 0.22.2 #439

aliciaolivaresgil opened this issue Mar 19, 2024 · 0 comments · Fixed by #444

Comments

@aliciaolivaresgil
Copy link

Describe the bug

In scikit-survival/sksurv/svm/survival_svm.py:

image

'cosine' kernel is not included in the 'kernel' options, but it is still described in the documentation.

Versions
image
@aliciaolivaresgil aliciaolivaresgil changed the title 'cosine' kernel in FastKernelSurvivalSVM still in documentation but not working in 0.22.0 'cosine' kernel in FastKernelSurvivalSVM still in documentation but not working in 0.22.2 Mar 19, 2024
sebp added a commit that referenced this issue Mar 20, 2024
@sebp
Fix allowed values for kernel
5018f92 
FastSurvivalSVM and MinlipSurvivalAnalysis

Closes #439
sebp added a commit that referenced this issue Apr 2, 2024
@sebp
Fix allowed values for kernel
6984da2 
FastSurvivalSVM and MinlipSurvivalAnalysis

Closes #439
@sebp sebp mentioned this issue Apr 2, 2024
Merged
5 tasks
@sebp sebp closed this as completed in #444 Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix allowed values for kernel sebp/scikit-survival
1 participant
@aliciaolivaresgil