ROS ov2.0 by ChloeWu1 · Pull Request #195 · intel/ros_openvino_toolkit

ChloeWu1 · 2022-11-02T09:44:27Z

No description provided.

…ject_detection_ssd model

huangjiafengx · 2022-12-09T03:42:50Z

data/model_list/download_model.lst

+# This file can be used with the --list option of the model downloader.
+face-detection-adas-0001
+age-gender-recognition-retail-0013
+emotions-recognition-retail-0003


no found the xml: face-reidentification-retail-0095.xml
if it is need.

and the xml vehicle-license-plate-detection-barrier-0123.xml

huangjiafengx · 2022-12-09T05:28:07Z

docker/Dockerfile

+&& rm -rf /var/lib/apt/lists/*
+
+# other dependencies
+RUN apt-get update && apt-get install -y python3-pip && python3 -m pip install -U \


the libcabera-gtk* would be install, else the pipeline_face_reidentification.launch would be fail.
apt-get install libcanberra-gtk*

huangjiafengx · 2022-12-09T07:01:39Z

docker/docker_instruction.md

+* Before launch, check the parameter configuration in ros_openvino_toolkit/sample/param/xxxx.yaml, make sure the paramter like model path, label path, inputs are right.
+  * run face detection sample code input from StandardCamera.
+  ```
+  roslaunch vino_launch pipeline_people.launch


According the ducument the path such as xml image and labels etc in the yaml file that need your adjustment like ros2_openvino_toolkit.

huangjiafengx · 2022-12-09T08:48:18Z

doc/quick_start/getting_started_with_Noetic_Ubuntu20.04_ov2.0.md

+ sudo cp ~/catkin_ws/src/ros_openvino_toolkit/data/labels/emotions-recognition/FP32/emotions-recognition-retail-0003.labels /opt/openvino_toolkit/models/emotions-recognition/output/intel/emotions-recognition-retail-0003/FP16/
+ sudo cp ~/catkin_ws/src/ros_openvino_toolkit/data/labels/face_detection/face-detection-adas-0001.labels /opt/openvino_toolkit/models/face_detection/output/intel/face-detection-adas-0001/FP32/
+ sudo cp ~/catkin_ws/src/ros_openvino_toolkit/data/labels/face_detection/face-detection-adas-0001.labels /opt/openvino_toolkit/models/face_detection/output/intel/face-detection-adas-0001/FP16/
+ sudo cp ~/catkin_ws/src/ros_openvino_toolkit/data/labels/object_detection/vehicle-license-plate-detection-barrier-0106.labels /opt/openvino_toolkit/models/vehicle-license-plate-detection/output/convert/intel/vehicle-license-plate-detection-barrier-0106/FP32


There is used FP16/vehicle-license-plate-detection-barrier-0106.labels at the pipeline_vehicle_detection.yaml. but there are FP32

huangjiafengx · 2022-12-09T08:56:02Z

vino_core_lib/include/vino_core_lib/models/attributes/base_attribute.h

  }

-  virtual bool updateLayerProperty(const InferenceEngine::CNNNetwork&)
+  virtual bool updateLayerProperty(std::shared_ptr<ov::Model>&)


If leave the const is better?

docker/Dockerfile

huangjiafengx · 2022-12-13T05:48:28Z

.vscode/settings.json

+        "variant": "cpp",
+        "bit": "cpp"
+    }
+}


No newline at the end.

huangjiafengx · 2022-12-15T04:26:20Z

data/model_list/download_model.lst

+semantic-segmentation-adas-0001
+mobilenet-ssd
+deeplabv3
+face-reidentification-retail-0095.xml


sorry, it should be
face-reidentification-retail-0095 and
vehicle-license-plate-detection-barrier-0123.

Fixed. Thanks.

LewisLiuPub

Please evaluate my comments and sync them with ROS2 version.

LewisLiuPub · 2022-12-23T01:05:03Z

docker/Dockerfile

+# ENV FTP_PROXY="your_proxy"
+
+# maintainer information
+LABEL maintainer="Jiawei Wu <jiawei.wu@intel.com>"


It's better to label the author info: LABEL author="Jiawei Wu jiawei.wu@intel.com"

LewisLiuPub · 2022-12-23T01:12:56Z

docker/Dockerfile

@@ -0,0 +1,82 @@
+# ros openvino toolkit env master ec5b9b9716e4
+
+ARG ROS_VERSION


rename ROS_VERSION -->ROS_PRE_INSTALLED_PKG?

LewisLiuPub · 2022-12-23T01:17:08Z

docker/Dockerfile

+
+ARG ROS_VERSION
+FROM osrf/ros:${ROS_VERSION}
+ARG VERSION


update these 3 lines above as:
ARG VERSION=noetic
ARG ROS_PRE_INSTALLED_PKG=${VERSION}-desktop-full
FROM osrf/ros:${ROS_PRE_INSTALLED_PKG}

LewisLiuPub · 2022-12-23T01:23:11Z

docker/docker_instruction.md

+```
+cd ~/ros_openvino_toolkit/docker/Dockerfile
+vi ~/ros_openvino_toolkit/docker/Dockerfile
+docker build --build-arg ROS_VERSION=<EXPECT_ROS_BASE_IMAGE> --build-arg VERSION=<EXPECT_ROS_VERSION> --build-arg "HTTP_PROXY=set_your_proxy" -t ros_openvino_202201 .


ROS_VERSION and VERSION, quite similar, rename it.

what if we always install xxxx-desktop-full? making ROS_VERSION optional in the command.

LewisLiuPub · 2022-12-23T01:37:42Z

docker/Dockerfile

+WORKDIR /root/catkin_ws/src
+RUN git init && git clone -b ros https://github.com/intel/ros_openvino_toolkit \
+&& git clone https://github.com/intel/object_msgs.git \
+&& git clone -b ${VERSION} https://github.com/ros-perception/vision_opencv.git \


cv_bridge and realsense-ros should be install by "apt install xxxx"

Removed. No need to install.

LewisLiuPub · 2022-12-23T01:47:38Z

docker/Dockerfile

+requests \
+&& apt-get install -y --no-install-recommends libboost-all-dev
+WORKDIR /usr/lib/x86_64-linux-gnu
+RUN ln -sf libboost_python-py36.so libboost_python37.so


Don't link between different versions.
We should find the root cause, then find the true implementation.

huangjiafengx · 2022-12-30T01:52:59Z

docker/Dockerfile

-&& git clone https://github.com/intel/object_msgs.git \
-&& git clone -b ${VERSION} https://github.com/ros-perception/vision_opencv.git \
-&& git clone -b ros1-legacy https://github.com/IntelRealSense/realsense-ros.git
+&& git clone https://github.com/intel/object_msgs.git


why remove the git of vision_opencv and realsense?

No dependency on this two package.

huangjiafengx · 2022-12-30T02:10:39Z

vino_launch/param/pipeline_vehicle_detection.yaml

+  input_path: to/be/set/video_path 
  infers:
    - name: ObjectDetection
      model: /opt/openvino_toolkit/models/public/vehicle-license-plate-detection-barrier-0123/FP16/vehicle-license-plate-detection-barrier-0123.xml


The model is vehicle-license-plate-detection-barrier-0123.xml, but label is vehicle-license-plate-detection-barrier-0106.labels

Same as ov2021.

ChloeWu1 added 25 commits October 10, 2022 13:24

fix engine api

ffa78bf

fix pipeline api

ac2e596

fix base_inference api

9dfd32f

fix version info

aed1c24

fix base model

04d7a0d

fix age gender detection pipeline

4921a24

fix emotion detection pipeline

0b224e2

fix head pose detection pipeline

451dfcd

fix person reidentification pipeline

f005a2f

fix object segmentation pipeline

4273a40

fix object detection(ssd) pipeline

6d5b7a0

fix license plate detection pipeline

853f773

fix vehicle attribs detection pipeline

daae598

fix person attribs detection pipeline

de67e59

fix object detection(yolo) pipeline

ef7c030

fix face reidentification pipeline

0d3d74a

fix landmarks detection pipeline

0060965

fix pipeline manager

b120aa0

fix pipeline params API

4eb2869

fix CMakeLists

439c569

clean code and fix head files

68a5aa4

Add NCHW and NHWC layout compatibility for object_segmentation and ob…

374b648

…ject_detection_ssd model

clean code

ca444e6

update docs for ov2.0

dc8e996

fix docs

b79a632

ChloeWu1 changed the title ~~Noetic ov2022.1~~ ROS ov2。0 Nov 2, 2022

ChloeWu1 changed the title ~~ROS ov2。0~~ ROS ov2.0 Nov 2, 2022

ChloeWu1 added 3 commits November 11, 2022 11:01

fix param name

c6f4fa3

code refine: move input&output info definition to base attribute

7adb00c

update doc and model list for ros

5ad3792

ChloeWu1 added 5 commits December 8, 2022 13:49

add dockerfile and instruction for ros

474c8dd

refine dockerfile

a479937

add segmentation maskrcnn pipeline

fed6345

fix vehicle attribs and license plate

3c23d15

fix yaml path

1158e5c

huangjiafengx reviewed Dec 13, 2022

View reviewed changes

ChloeWu1 added 2 commits December 13, 2022 14:24

fix doc dockerfile and model list

cfea37d

fix param

3d27120

huangjiafengx reviewed Dec 15, 2022

View reviewed changes

ChloeWu1 added 4 commits December 15, 2022 12:58

fix model download list

809ebfb

fix label path

3d97a61

fix label path for docker

b7f6ee6

update documents for ros 2.0

98fb7ce

LewisLiuPub reviewed Dec 23, 2022

View reviewed changes

ChloeWu1 added 2 commits December 23, 2022 11:12

Merge branch 'ros' into noetic-ov2022.1

066ab65

fix docker

fad9c8b

huangjiafengx reviewed Dec 30, 2022

View reviewed changes

		@@ -0,0 +1,82 @@
		# ros openvino toolkit env master ec5b9b9716e4

		ARG ROS_VERSION

Conversation

ChloeWu1 commented Nov 2, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LewisLiuPub left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants