-
Notifications
You must be signed in to change notification settings - Fork 87
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
Closes #3665: add multi-dim support to arkouda.testing module #3751
Closes #3665: add multi-dim support to arkouda.testing module #3751
Conversation
1606baa
to
59467cf
Compare
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 left two comments on the first of the tests, but it seems to apply to the others as well
3cc1e3b
to
8837809
Compare
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.
Looks good
5c88fbb
to
94ec552
Compare
6075d90
to
3be21e8
Compare
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.
looks great!
Multi-dim seems to be working, resulting from the other recent PRs. This ticket verifies it's working by adding some multi-dim unit tests.
Closes #3665: add multi-dim support to arkouda.testing module