-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
API: Update lib.stride_tricks
namespace
#24580
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
Conversation
lib.stride_tricks
namespace
e275ece
to
8de5f9a
Compare
I see
So IMO, we should make it private and add a note to the migration guide that downstream projects who need it should just copy/paste it into their source tree. There's no need to expand the docstrings or make public something so trivial to implement. |
8de5f9a
to
6e12461
Compare
That makes sense to me. In the github search engine it shows only 13 hits, so I removed it from the public |
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, are there any downstream usages that need to get resolved before this can be merged?
This comment was marked as outdated.
This comment was marked as outdated.
That link doesn't show me anything. The |
Oops, sorry, you're right, they're all usages of |
OK, in that case, in it goes. |
Relevant issue #24507
Hi @rgommers @ngoldbaum,
This PR moves
lib.stride_tricks
module to a private file and ensures that it's public methods are only available through the main or local namespaces. For local namespace there areas_strided
andsliding_window_view
functions.