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

Modernize Python pipeline scripts and add optional parameters. #2217

Open

yanxke wants to merge 3 commits into openMVG:develop from yanxke:yan/argparse

Contributor

yanxke commented Jul 5, 2023

No description provided.

yanxke added 2 commits

July 5, 2023 20:10


          Modernize Python pipeline scripts and add optional parameters.

cc30f7e


          Fix str in args.

7274d3a

Member

pmoulon commented Jul 6, 2023

Since it changes how the binary are called we would also have to update the documentation
See here https://github.com/openMVG/openMVG/blob/develop/docs/sphinx/rst/software/SfM/SfM.rst#openmvg-sfm-pipelines-demo


          Add documentation.

0a53923

Contributor Author

yanxke commented Jul 7, 2023

Sure, added. The default behavior is the same as before as the new parameters are optional.

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