Skip to content
This repository has been archived by the owner on Aug 7, 2024. It is now read-only.

Adds human pose estimation #177

Open
wants to merge 20 commits into
base: dev-ov2020.3
Choose a base branch
from
Open

Adds human pose estimation #177

wants to merge 20 commits into from

Conversation

silasalves
Copy link

@silasalves silasalves commented Jun 30, 2021 •
edited
Loading

Adds support to human-pose-estimation-0001 network to the new development branch, as suggested in the previous pull request.

chaoli2 and others added 20 commits May 5, 2019 10:53
@chaoli2
Merge pull request intel#35 from intel/devel
987c926 
merge devel to master for 0.5.0 release
@chaoli2
fix README and config issues
7d7aa11 
Signed-off-by: Chao Li <[email protected]>
@chia-labradorsystems
add object_msgs in find_package
9134114 
to solve CMake Error at /opt/ros/kinetic/share/genmsg/cmake/genmsg-extras.cmake:263 (message):
  Messages depends on unknown pkg: object_msgs (Missing
  'find_package(object_msgs)'?)
@LewisLiuPub
Merge pull request intel#57 from chia-labradorsystems/patch-1
0307e6b 
add  object_msgs in find_package
@sradmard
people_msgs package name is changed to vino_people_msgs to avoid conf…
5ae6157 
…licting name with another ros package
@LewisLiuPub
Merge pull request intel#64 from sradmard/master
0e98304 
people_msgs package name is changed to vino_people_msgs
@sradmard
dynamic_vino_sample package is renamed to vino_sample for naming cons…
ac2a883 
…istency
@sradmard
dynamic_vino_lib is renamed to vino_core_lib for naming consistency
448992a
@LewisLiuPub
Merge pull request intel#67 from sradmard/master
5cf4984 
Name Changes for sample and dynamic_vino_lib packages.
@silasalves
Adds model and inference for pose estimation.
bc303bc
@silasalves
Implements the human pose detection.
b8bf41d
@silasalves
Adds ROS topic output for human pose estimation.
b1c2aac
@silasalves
Adds visualization output.
8f882f2
@silasalves
Adds RViz support.
a5d6d02
@silasalves
Fix keypoint replication bug.
8206e0d
@silasalves
Adds ROS service support for human pose estimation
7f212a8
@silasalves
Adds missing model verification.
041534e
@silasalves
Fixes bugs and adds doc and parameter support.
bb1ca3f
@silasalves
Adds scores to individual keypoints.
8f5464b
@silasalves
Fix ROS message.
e8bb2bc
@LewisLiuPub
Copy link
Contributor

@silasalves
Thank you very much for your quick response.
There are some conflicts between your patches and the original code of the branch, mostly because of the naming changes in branch master.

To simplify the PR's merge, do you think it would be a good idea to merge Master's new patches to this branch?

Thanks a lot.

@LewisLiuPub
Copy link
Contributor

oh, you have already merged master's new patches to this branch, that's good. Thanks. @silasalves

@Corsair-cxs, since you are now contributing to this branch, please pay attention to the changes in this PR.

@Corsair-cxs Corsair-cxs mentioned this pull request Jul 4, 2021
Open
13 tasks
@LewisLiuPub
Copy link
Contributor

@silasalves
Hi, Silas, per our check, the patches you integrated into Branch dev-ov2020.3 can't pass build, because of some old code of OpenVINO and ROS versions.
I am wondering if we can restructure your contributions into 2 phase:

  1. Enable Human pose feature. (@Corsair-cxs has already published one PR: Adds human pose estimation support #178 , I will add you in the contributor list, if you agree.)
  2. Change folder names following the way you did, after the Human pose feature passes the func test based on OpenVINO 2020.3LTS version.

Please let me know if you have different points.
Thanks.

@LewisLiuPub LewisLiuPub mentioned this pull request Jan 25, 2022
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@silasalves @LewisLiuPub @chaoli2 @chia-labradorsystems @sradmard