Skip to content
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

bugfix in video demo and feature in visualization #5

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

bugfix in video demo and feature in visualization #5

wants to merge 10 commits into from

Conversation

TheTesla
Copy link
Contributor

bugfix: autodetect opencv major version - variable is string must be converted in int;
feature: RGB->BGR convertion optional in plot_pose_stickmodel_cv2mat()

@pulkitag
Copy link
Owner

I will have a look at it this week. Sorry for the delay in response.

… reuse for same person and csv person number; added frame number to csv; parameters about human detection
@pulkitag
Copy link
Owner

pulkitag commented Apr 1, 2016

Hi Stefan,

A couple of things before I merge this:

  1. Can you make the python code so that each line is atmost 80 characters long? (i.e. break long lines into multiple lines). This makes code more readable.
  2. In the class PoseIEF - can you please include a short description of the new functions and how they should be used?
  3. Regarding naming convention: Lets follow this to be consistent with rest of the code (https://github.com/pulkitag/ief/wiki/Contributing)

@TheTesla
Copy link
Contributor Author

I am doing the work to prepare my contributions for your needs. Should I use 4 or 8 space for indentation? Will you provide the code for training?

@pulkitag
Copy link
Owner

4 space. Hoping to release the training code in Dec.

On Oct 14, 2016 3:06 PM, "Stefan Helmert" [email protected] wrote:

I am doing the work to prepare my contributions for your needs. Should I
use 4 or 8 space for indentation? Will you provide the code for training?


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#5 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AETXxoq8dCmRpSnQf3M7UHHZKS71uK5Uks5qz4xSgaJpZM4Httpd
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants