[face_detection] Fix arg condition when launching with opencv4 in noetic

[tkmtnt7000]

On noetic with opencv4, the following error occurs because arg face_cascade_name declares twice.

$ roslaunch opencv_apps face_detection.launch use_opencv4:=true image:=/usb_cam/image_raw debug_view:=false 
... logging to /home/tsukamoto/.ros/log/816f6d2e-9c7e-11ed-a041-270d6101f3c5/roslaunch-tsukamoto-desktop-ryzen-471330.log
Checking log directory for disk usage. This may take a while.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.

RLException: Invalid <arg> tag: arg 'face_cascade_name' has already been declared. 

Arg xml is <arg unless="$(arg use_opencv3)" name="face_cascade_name" default="/usr/share/opencv/haarcascades/haarcascade_frontalface_alt.xml" doc="Face dtection cascade Filename"/>
The traceback for the exception was written to the log file

I think I fixed it in this PR, but this may not be the best solution because it contains breaking changes.
So, could you tell me if anyone have another solution?

And I also fixed typo.

[k-okada]

thank you. can you add test code for opencv4? https://github.com/ros-perception/opencv_apps/blob/74fad3a6f7d52ee6de61cfc1461fdfaebf12534f/test/CMakeLists.txt#L83-L86 -- ◉ Kei Okada 2023年1月25日(水) 18:46 Naoto Tsukamoto ***@***.***>:

…

On noetic with opencv4, the following error occurs because arg face_cascade_name declares twice. $ roslaunch opencv_apps face_detection.launch use_opencv4:=true image:=/usb_cam/image_raw debug_view:=false ... logging to /home/tsukamoto/.ros/log/816f6d2e-9c7e-11ed-a041-270d6101f3c5/roslaunch-tsukamoto-desktop-ryzen-471330.log Checking log directory for disk usage. This may take a while. Press Ctrl-C to interrupt Done checking log file disk usage. Usage is <1GB. RLException: Invalid <arg> tag: arg 'face_cascade_name' has already been declared. Arg xml is <arg unless="$(arg use_opencv3)" name="face_cascade_name" default="/usr/share/opencv/haarcascades/haarcascade_frontalface_alt.xml" doc="Face dtection cascade Filename"/> The traceback for the exception was written to the log file I think I fixed it in this PR, but this may not be the best solution because it contains breaking changes. So, could you tell me if anyone have another solution? And I also fixed typo. ------------------------------ You can view, comment on, or merge this pull request online at: #142 Commit Summary - a620ceb <a620ceb> Fix arg condition when launching with opencv4 - 0ff70d5 <0ff70d5> Fix typo; dtection=>detection File Changes (1 file <https://github.com/ros-perception/opencv_apps/pull/142/files>) - *M* launch/face_detection.launch <https://github.com/ros-perception/opencv_apps/pull/142/files#diff-36d442b541c96cb161da3972a5b67f3aed59dbb4d09ce4a53a9c5bb5e914e4d5> (26) Patch Links: - https://github.com/ros-perception/opencv_apps/pull/142.patch - https://github.com/ros-perception/opencv_apps/pull/142.diff — Reply to this email directly, view it on GitHub <#142>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADYNXGAFJG6KUPGJTMRLVTWUDY5RANCNFSM6AAAAAAUGDB6NQ> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>