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.

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

[BUG][WIP] New scikit-learn versions and warnings #154

Merged
merged 15 commits into from
Jun 17, 2022

Conversation

samihamdan
Copy link
Collaborator

#152
Also removing other warnings and errors in the tests due to newer sklearn versions

@samihamdan
Copy link
Collaborator Author

I updated julearn to run with newer version of sklearn.
also dropped very old support.

@samihamdan
Copy link
Collaborator Author

There is a discussion we have to have about what behavior really makes sense for y_transforming into different problem_types etc. This is from what I see resulting in failing tests. Should one really be able to apply zscoring on label data ?

@samihamdan samihamdan changed the title Feature Names Warning when using Confound Removal [BUG][WIP] New scikit-learn versions and warnings Jun 14, 2022
@samihamdan
Copy link
Collaborator Author

I am only supporting now up to 1.0 for all things.
We will have to look at compatibility of target transformation with changing the problem_type for sklearn==1.1.
This means for some edge cases we do not support sklearn >1.0 at the moment.
But tbh I think no own will use the non supported features before we can fix them as well for 1.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants