davidpagnon/Sports2D

Colab wrong command?

p0mad opened this issue · 5 comments

p0mad commented

Hi @davidpagnon
I've checked your colab notebook placed here
and i found that it might have some wrong command! (not sure if the openpose is checkedout to specific tag or not)

!sed -i 's/execute_process(COMMAND git checkout master WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}/3rdparty/caffe)/execute_process(COMMAND git checkout f019d0dfe86f49d1140961f8c7dec22130c83154 WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}/3rdparty/caffe)/g' openpose/CMakeLists.txt

This line of code do noting actually!

here is my clone of openpose (today) without checkout, and i can see that openpose/CMakeLists.txt containes lines with git checkout master but all of them are commented out!
image

either this command is not needed or needs to be modified for replacing the 1807aad with f019d0d for caffe!

Also does it needs to install opencv as a requirement?( i think the openpose > V1.6.0 do not needs the opencv while with the available code on a native ubuntu system, it askes for opencv while making it with the default CMakeList.txt file!

Thanks
Best regards

p0mad commented

This is the cmake error
image

have you check out into specific version of openpose?

If I understand well, you are trying to run the Colab code on a local Ubuntu station?

While both setups are quite similar, a standard Ubuntu installation does not come with as many libraries.

For example, OpenCV is natively installed on Colab and I did not bother install it again. You will likely need to install it on your computer (see the paragraphs coming right after your make error for indications about how to do that).

To be honest, I took most of the code from there and adapted it because it was buggy. I'm not an expert in building project but I'm pretty sure the line of code you're referring to is replacing master with the specific version f019d0dfe86f49d1140961f8c7dec22130c83154 in the CMakeLists.txt file.

p0mad commented

I'm pretty sure the line of code you're referring to is replacing master with the specific version f019d0dfe86f49d1140961f8c7dec22130c83154 in the CMakeLists.txt file

This defenately replaces master with f019d0 but as the line including "maste" is commented out, So it does notting! you can ommit this line and test the code!

Thanks for refering to the original code

I've also installed opencv but make -jnproc troughs an error about boost library!
Thats annoying!

Okay I understand what you mean. Did you try replacing 1807aad with the other version? The other version has been tested and works, but apparently it is not needed since it made a change on a commented line.

Concerning the Opencv issue, I am sorry, I don't have a lot of time this week, and I'm not even sure I can help you since I don't have a lot of experience in building with Cmake. I usually just try to change little things until it works, which is definitely not the best way to proceed...

Do you have anyone in your team that could help you? If not, hit me up in a little while (and if so, maybe you could just use the Colab version, which works?)

Hi @p0mad ,
(very) long story short, the colab version should be up now! Don't hesitate to tell me if you face any issues.
I'm closing this for now but if you experience any problems, please reopen the issue.