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

Add tests for slice_along_axis and sliding_win_oneaxis. #104

Merged
merged 1 commit into from
Mar 4, 2024

Conversation

cboulay
Copy link
Collaborator

@cboulay cboulay commented Mar 4, 2024

(I swear I made this PR already...)

This just updates some docstrings and adds a couple unit tests for some ndarray utility functions I made. There are no functional changes here.

Typically, I would prefer to wait for review even if a PR only touches unit tests because there might be some misunderstanding about intention and scope. However, given that the tests are for functions that I wrote recently and no one has really had a chance to use yet, I am confident that I can merge this PR without causing any problems.

@cboulay cboulay changed the base branch from main to dev March 4, 2024 17:12
@cboulay cboulay merged commit ba6f50b into dev Mar 4, 2024
9 checks passed
@cboulay cboulay deleted the cboulay/ndarray_util_tests branch March 5, 2024 22:45
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.

1 participant