Skip to content

alternative implementation of dpnp.nanmedian #2213

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

alternative implementation of dpnp.nanmedian #2213

wants to merge 2 commits into from

Conversation

vtavana
Copy link
Contributor

@vtavana vtavana commented Dec 4, 2024

This is alternative approach for implementing nanmedian. The first implementation is available at #2191

  • Have you provided a meaningful PR description?
  • Have you added a test, reproducer or referred to issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • Have you checked performance impact of proposed changes?
  • If this PR is a work in progress, are you filing the PR as a draft?

@vtavana vtavana self-assigned this Dec 4, 2024
@vtavana vtavana mentioned this pull request Dec 4, 2024
Merged
6 tasks
@vtavana
Copy link
Contributor Author

vtavana commented Dec 4, 2024

The timing for this implementation is similar to other implementation in #2191. Detailed comparison is added to the other PR. Closing this one.

@vtavana vtavana closed this Dec 4, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 4, 2024 •
edited
Loading

View rendered docs @ https://intelpython.github.io/dpnp/index.html

@antonwolfy antonwolfy deleted the nanmedian-numpy branch May 14, 2025 16:41
@vtavana vtavana restored the nanmedian-numpy branch July 21, 2025 18:09
@vtavana vtavana deleted the nanmedian-numpy branch July 21, 2025 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@antonwolfy antonwolfy Awaiting requested review from antonwolfy antonwolfy will be requested when the pull request is marked ready for review antonwolfy is a code owner

@vlad-perevezentsev vlad-perevezentsev Awaiting requested review from vlad-perevezentsev vlad-perevezentsev will be requested when the pull request is marked ready for review vlad-perevezentsev is a code owner

@ndgrigorian ndgrigorian Awaiting requested review from ndgrigorian ndgrigorian will be requested when the pull request is marked ready for review ndgrigorian is a code owner

Assignees

@vtavana vtavana

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vtavana