search

sthoduka / kortex_vision

non-ROS library for KINOVA® KORTEX™ arms vision module
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

Suggestion: argparser for main #3

Closed lscarton closed 5 months ago

lscarton commented 6 months ago

I suggest to use OpenCV cv::CommandLineParser as argparser for the main function.

https://docs.opencv.org/3.4/d0/d2e/classcv_1_1CommandLineParser.html

lscarton commented 6 months ago

@sthoduka The argparser is now done, could you please see if it is similar to what you had in mind?

lscarton commented 6 months ago

the command ./KinovaVisionMain -h currently shows:

Non-ROS library for KINOVA® KORTEX™ arms vision module.
Usage: KinovaVisionMain [params] 

        -c, --config (value:../config/config.json)
                Usage: Path to json config file.
        --depthresolution (value:424x240)
                Available: 424x240, 480x270
        -h, --help (value:true)
                Print help message.
        --rgbresolution (value:320x240)
                Available: 320x240, 640x480, 1280x720, 1920x1080
        --use_depth (value:true)
                Usage : enable the use of Depth camera.
        --use_rgb (value:true)
                Usage: enable the use of RGB camera.
lscarton commented 6 months ago

It is unfortunately in alphabetical order

sthoduka commented 6 months ago

I think this looks good; these are the options that someone would want to change once the config file has been fixed.