-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
MAINT Add _distributor_init.py #15570
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
MAINT Add _distributor_init.py #15570
Conversation
NumPy has `_distributor_init.py`: https://github.com/numpy/numpy/blob/master/numpy/_distributor_init.py This PR adds one for scikit-learn. The intent is to create a place to add distributor specific logic, e.g. to implement behavior like in scikit-learn#14247 (comment)
ping. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is an acceptable solution if we are going to acknowledge the place of platform-specific forks and want to keep them in sync with our releases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am okay with this.
@rth are we okay to merge? Does your addition of Needs Decision imply that we cannot rely on two approvals? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM as well.
Does your addition of Needs Decision imply that we cannot rely on two approvals?
No I didn't put "Waiting for Reviewer" because I though it needed more a decision than a review, but I probably should have.
Thanks @oleksandr-pavlyk ! We should backport this to 0.23.X so it could be included in the release.. |
NumPy has
_distributor_init.py
:https://github.com/numpy/numpy/blob/master/numpy/_distributor_init.py
This PR adds one for scikit-learn. The intent is to create a place
to add distributor specific logic, e.g. to implement behavior like in
#14247 (comment)
Reference Issues/PRs
What does this implement/fix? Explain your changes.
Any other comments?