-
Notifications
You must be signed in to change notification settings - Fork 22
Statistics funcs cov #2
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
@Rubtsowa please save original paths, file names and file contents (as it possible) when you add the tests. Please look at these changes for better explanation what I mean. |
cpdef dparray dpnp_cov(dparray array1): | ||
# behaviour of original numpy | ||
if array1.ndim > 2: | ||
raise ValueError("array has more than 2 dimensions") |
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.
use function from utils
|
||
is_dparray1 = isinstance(in_array1, dparray) | ||
|
||
if (not use_origin_backend(in_array1) and is_dparray1): |
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.
👍
The PR proposes to add fixes/improvements into the public CI: - trigger resetup miniconda step on failure in building docs and coverage workflows - trigger resetup miniconda & reinstall dpnp steps in case of failure within `test_linux` job of conda package workflow Note, the PR continues work initiated in #2254 and indented to make public CI more stable.
The PR proposes to add fixes/improvements into the public CI: - trigger resetup miniconda step on failure in building docs and coverage workflows - trigger resetup miniconda & reinstall dpnp steps in case of failure within `test_linux` job of conda package workflow Note, the PR continues work initiated in #2254 and indented to make public CI more stable. 9d445ba
Naive implementation np.cov with using of already implemented another functions is added.
Tests for statistics functions are added, all that are available in statistics_tests except few specific (cupy.core._accelerator).
Paths and file names are changed to be the same as in original test suite, test file for numpy.mean is replaced by original.
3 test for 'mean' are passed as it was before changes.
+2 passed tests from 'histogram'.
+2 passed tests from 'correlation' (np.cov).