Connect all the libraries to main.cpp (i.e. build a proper CMakeLists.txt)

changh95 commented 3 years ago

We made some useful scripts to clone and build useful libraries, but we have not yet made any connections to the C++ source codes.

We need to:

Check the downloaded libraries are in correct format.
- We currently have build folder and source folder separately. On the other hand, some libraries recommend to make the build folders inside the source folder in their tutorials. We need to decide which structure we will use.
Check the downloaded libraries work as intended.
Decide on how we will connect the libraries.
- As modules? Extra cmake config files? etc.
Write CMakeLists.txt to connect the libraries.

Tracker page

[x] OpenCV - #5 #6 #8
[x] PCL - #7 #8
[x] Eigen #12
[x] Ceres-solver #12
[x] GTSAM #12
[ ] OpenGL
[x] spdlog #12
[ ] GTest

changh95 commented 3 years ago

Pinging @pacientes to share the content update :smile_cat:

pacientes commented 3 years ago

Support OpenCV and test done

changh95 commented 3 years ago

@pacientes 님,

저 라이브러리들을 모두 하나의 CMakeLists.txt에 링크하게 되면 많이 길어질 것 같다는 생각이 듭니다. 저희 폴더 구조중에 /cmake폴더가 있는데, 각각의 모듈을 링크하는 코드 부분을 떼어놓을 수 있을까요?

changh95 commented 3 years ago

그리고 이번 #8 을 보시면 './third_party/{module}/install`에 최종 라이브러리가 저장되게 됩니다.

이렇게되면 각각 모듈의 소스폴더와 빌드폴더가 필요없을 수도 있을 것 같은데요. 용량을 아끼기 위해 삭제할 수 있을 것 같습니다. 대신 빌드 중 문제가 생겼을 때 처음부터 다시 시작해야하는 단점이 있을 것 같아요 ㅋㅋ (그 몇시간을 다시...)

이 부분을 자동으로 지워주는 스크립트를 짜는거에 대해서 어떻게 생각하시나요?

pacientes commented 3 years ago

@pacientes 님,

저 라이브러리들을 모두 하나의 CMakeLists.txt에 링크하게 되면 많이 길어질 것 같다는 생각이 듭니다. 저희 폴더 구조중에 /cmake폴더가 있는데, 각각의 모듈을 링크하는 코드 부분을 떼어놓을 수 있을까요?

네 가능합니다. 해당 부분 작업해서 PR 남겨놓을게요.

pacientes commented 3 years ago

그리고 이번 #8 을 보시면 './third_party/{module}/install`에 최종 라이브러리가 저장되게 됩니다.

이렇게되면 각각 모듈의 소스폴더와 빌드폴더가 필요없을 수도 있을 것 같은데요. 용량을 아끼기 위해 삭제할 수 있을 것 같습니다. 대신 빌드 중 문제가 생겼을 때 처음부터 다시 시작해야하는 단점이 있을 것 같아요 ㅋㅋ (그 몇시간을 다시...)

이 부분을 자동으로 지워주는 스크립트를 짜는거에 대해서 어떻게 생각하시나요?

make 산출물과 make install 산출물이 각각 다른데요. make 산출물을 삭제할 경우 용량확보가 가능하지만, make uninstall 같은 명령어를 사용할 수 없는 단점이 있습니다.

하지만, make install의 경로가 써드파티 디렉토리에 생기면서, 나중에 라이브러리 문제가 생길 경우 해당 디렉토리를 전부 삭제하고 다시 설치하면 되는 것이라! 용량 확보를 위해 삭제 스크립트가 있는 것이 좋아보입니다!

changh95 commented 3 years ago

PR #12로 인한 Eigen, ceres-solver, GTSAM, spdlog 설정 완료 사항 업데이트 했습니다!

changh95 commented 3 years ago

v0.5에서는 여기까지만 하고 릴리즈 하겠습니다!

끝내지 못한 OpenGL, GTest, 그리고 PCL에 VTK 옵션 추가 등등은 v1.0에서 하면 좋을 것 같습니다! 의견 없으시면 이 이슈는 닫겠습니다 :smile_cat:

changh95 commented 3 years ago

v0.5까지 작업하기로 합의한 내용들은 전부 완료되었습니다. 추후 작업할 내용들은 #17 과 #3 로 옮겼습니다.

changh95 / cpp-cv-project-template

Connect all the libraries to main.cpp (i.e. build a proper CMakeLists.txt) #2

Tracker page