Skip to content
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

Add gdb pretty printer for cv::Mat #20547

Merged
merged 2 commits into from
Aug 17, 2021
Merged

Add gdb pretty printer for cv::Mat #20547

merged 2 commits into from
Aug 17, 2021

Conversation

rogday
Copy link
Member

@rogday rogday commented Aug 12, 2021

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 other license that is incompatible with OpenCV
  • The PR is proposed to proper branch
  • There is reference to original 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

sergregory
sergregory reviewed Aug 12, 2021
View reviewed changes
samples/gdb/mat_pretty_printer.py Outdated Show resolved Hide resolved
samples/gdb/mat_pretty_printer.py Outdated Show resolved Hide resolved
samples/gdb/mat_pretty_printer.py Show resolved Hide resolved
@rogday rogday force-pushed the gdb_pretty_printer branch 2 times, most recently from 0cb1025 to 4b8e41c Compare August 16, 2021 10:03
@sergregory
Copy link

LGTM
@asmorkalov , your turn

@asmorkalov
Copy link
Contributor

@alalek Could you merge the PR?

@alalek alalek merged commit a50dec8 into opencv:master Aug 17, 2021
@rogday rogday deleted the gdb_pretty_printer branch October 7, 2021 13:15
@alalek alalek mentioned this pull request Oct 15, 2021
Merged
a-sajjad72 pushed a commit to a-sajjad72/opencv that referenced this pull request Mar 30, 2023
@rogday
Merge pull request opencv#20547 from rogday:gdb_pretty_printer
b30630c 
* add gdb rpetty printer for cv::Mat

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

@sergregory sergregory sergregory approved these changes

Assignees

@sergregory sergregory

Labels
category: documentation Documentation fix or update category: samples
Projects
None yet
Milestone
4.5.4
Development

Successfully merging this pull request may close these issues.
4 participants
@rogday @sergregory @asmorkalov @alalek