Skip to content

Make stitching script more pythonic#16257

Merged
opencv-pushbot merged 1 commit intoopencv:masterfrom
ianare:more-pythonic
Jan 16, 2020
Merged

Make stitching script more pythonic#16257
opencv-pushbot merged 1 commit intoopencv:masterfrom
ianare:more-pythonic

Conversation

@ianare
Copy link
Copy Markdown
Contributor

@ianare ianare commented Dec 29, 2019

This pullrequest changes

Improve the overall readability and maintainability of the Python example stitching script.

Specifically:

  • PEP8 fixes
  • Removed unused variables
  • Start splitting main function into easier to understand functions.
  • Use dictionaries for option parsing rather than if, this also allows easily showing all available options in the help output.
  • Update help text (copied from C++ example)
  • Added the AKAZ and BRISK feature finders as options.

parser.add_argument(
'--timelapse', action='store', default=None,
help="Output warped images separately as frames of a time lapse movie, "
"with "fixed_" prepended to input file names.",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Non-escaped quotes here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks, force pushed some fixes.

Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@alalek
Copy link
Copy Markdown
Member

alalek commented Jan 13, 2020

/cc @LaurentBerger

@asmorkalov
Copy link
Copy Markdown
Contributor

@mshabunin @alalek Do you have any concerns about the patch?

@opencv-pushbot opencv-pushbot merged commit e4a6476 into opencv:master Jan 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants