Skip to content

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

Merged
merged 4 commits into from
Sep 7, 2020
Merged

Statistics funcs cov #2

merged 4 commits into from
Sep 7, 2020

Conversation

Alexander-Makaryev
Copy link
Contributor

@Alexander-Makaryev Alexander-Makaryev commented Sep 7, 2020

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).

@Alexander-Makaryev
Copy link
Contributor Author

@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")
Copy link
Contributor

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):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@shssf shssf merged commit 174c325 into master Sep 7, 2020
@shssf shssf deleted the statistics-funcs-cov branch September 7, 2020 13:31
AlexanderKalistratov added a commit to AlexanderKalistratov/dpnp that referenced this pull request Nov 12, 2024
AlexanderKalistratov added a commit to AlexanderKalistratov/dpnp that referenced this pull request Dec 4, 2024
AlexanderKalistratov added a commit to AlexanderKalistratov/dpnp that referenced this pull request Dec 4, 2024
github-actions bot added a commit that referenced this pull request Jan 13, 2025
antonwolfy added a commit that referenced this pull request Jan 13, 2025
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.
github-actions bot added a commit that referenced this pull request Jan 13, 2025
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
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.

2 participants