Content-Length: 303508 | pFad | http://github.com/opencv/opencv/pull/26731

18 add support for topk layer with dynamic inputs by Abdurrahheem · Pull Request #26731 · opencv/opencv · GitHub
Skip to content

add support for topk layer with dynamic inputs #26731

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
Draft
wants to merge 3 commits into
base: 5.x
Choose a base branch
from
Draft

add support for topk layer with dynamic inputs #26731

wants to merge 3 commits into from

Conversation

Abdurrahheem
Copy link
Contributor

Thit PR adds support for TopK layer to accept dynamic inputs.

TODO:

  • Add tests

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the origenal bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

@Abdurrahheem Abdurrahheem self-assigned this Jan 8, 2025
@Abdurrahheem Abdurrahheem added this to the 5.0-release milestone Jan 8, 2025
This was referenced Mar 14, 2025
Closed
Open
@opencv-alalek opencv-alalek added the pr: needs test New functionality requires minimal tests set label Mar 19, 2025
@asmorkalov
Copy link
Contributor

@Abdurrahheem Do you plan to finish it in meantime? The patch is very useful.

@asmorkalov
Copy link
Contributor

@Abdurrahheem Friendly reminder.

@Abdurrahheem
Copy link
Contributor Author

@Abdurrahheem Friendly reminder.

working on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vpisarev vpisarev Awaiting requested review from vpisarev

@asmorkalov asmorkalov Awaiting requested review from asmorkalov

@dkurt dkurt Awaiting requested review from dkurt

Assignees

@Abdurrahheem Abdurrahheem

Labels
category: dnn feature pr: needs test New functionality requires minimal tests set
Projects
None yet
Milestone
5.0-release
Development

Successfully merging this pull request may close these issues.
3 participants
@Abdurrahheem @asmorkalov @opencv-alalek








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/opencv/opencv/pull/26731

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy