-
Notifications
You must be signed in to change notification settings - Fork 82
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.
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
body tracking implementation #39
base: develop
Are you sure you want to change the base?
Conversation
Release GIL in device_get_capture
Fixes, Example
# Conflicts: # example/threads.py # pyk4a/pyk4a.cpp
Reformat code Add type hintings Fix code errors
Refactor tests. Patch module for tests.
# Conflicts: # README.md # pyk4a/pyk4a.cpp
Refactor tests
Any tentative release date for this? |
No sorry.
It used to work but now it would need a couple hours of refactor to be
updated.
Is this a feature you would use?
Would you be willing to help me test if I work on it?
…On Sun., Dec. 20, 2020, 15:21 Romil Punetha, ***@***.***> wrote:
Any tentative release date for this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#39 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABTB77F7EUWCBDMIXIY3RODSVZMELANCNFSM4PXAI55Q>
.
|
Yes, I have a kinect device and am looking at making use of the body tracking sdk using python. I can help test it. |
Codecov Report
@@ Coverage Diff @@
## develop #39 +/- ##
===========================================
- Coverage 91.55% 90.90% -0.65%
===========================================
Files 5 5
Lines 308 319 +11
===========================================
+ Hits 282 290 +8
- Misses 26 29 +3
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@romilpunetha it works again! For details, see property And example/pose_display_2d.py |
Great! I'll give it a try now. |
I'm getting the following error in reinstalling:
Using the following command:
|
I also edited the setup.py to not depend on the body tracking flag. The setup completed. However, the example is throwing the following error:
|
The error you are getting is from wrong python code being used. Maybe you are reusing an older pyk4a version. Make sure you install with Be careful with the include paths. You are including the regular SDK files. We need them but also the body tracking sdk (it is possible they are already included). The following command is from the readme of the body tracking branch. It installs directly the code from the live branch. Notice the path is k4abt which stands for k4a body tracking:
|
This is the full command:
I'm getting the following error due to compiler issues...fixing it:
|
Yes, with your body tracking install paths
…On Wed., Dec. 23, 2020, 01:31 Romil Punetha, ***@***.***> wrote:
Do I need to update these paths?
/home/louis/p/pyk4a/k4abt/usr/include/
/home/louis/p/pyk4a/k4abt/usr/lib/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#39 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABTB77FZCAXKR2NJ6E4YE2DSWGFFJANCNFSM4PXAI55Q>
.
|
Had some issues with the nvidia library. Fixed those and it seems to be working now. |
Any way to run the same on ubuntu 18.04? I used the following command to install this on windows:
For ubuntu, this is what I've done for now:
However, when I try to run the examples, it gives the following error:
|
The command for linux is simpler:
But I'm getting this error now:
|
Solved this by disabling secure boot on ubuntu. |
Previous PR was a mess so let's continue here.
Implements body tracking with
#ifdef BODY_TRACKING
macros.See README.md for instructions on how to install.
Enabling body tracking this way is a hack but we don't need to maintain two repositories.
Is there any other simple way to do this?