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

Continuous integration and tensorflow_cc #97

Closed
PeterBowman opened this issue Sep 25, 2019 · 3 comments
Closed

Continuous integration and tensorflow_cc #97

PeterBowman opened this issue Sep 25, 2019 · 3 comments

Comments

@PeterBowman
Copy link
Member

PeterBowman commented Sep 25, 2019

PR #93 lacked CI support on Travis due to build timeouts (#93 (comment)), it takes way too long to compile this beast from sources.

Perhaps this is the perfect scenario for docker (#93 (comment) and next comments). If we could prepare a docker image in which every dependency, including tensorflow_cc, is installed and then pulled by Travis, build times should be considerably shorter. Use https://hub.docker.com/u/roboticslabuc3m to host said images (or rather docker run files, I'm still a docker rookie).

@davidvelascogarcia
Copy link

Maybe using tensorflow_cc docker image.
I´ve been doing some tests in tensorflowDetection2D repo using dockerimage. The configuration that i used was .travis.yml.
Using that configuration the script build and install yarp from source and install openCV. Required time take around 15-20 minutes by commit. Attached travis CI example using docker.

jgvictores added a commit that referenced this issue Nov 1, 2020
- Merge programs: #79
  - Old: colorRegionDetection2D + haarDetection2D
  - New: switchDetection2D + libs (HaarDetector + ColorRegionDetector)
- Via new API: #69
- Also new Tests: #99
- Integrated in CI: #97
@jgvictores
Copy link
Member

Dropping as wontfix in favor of #103, right @PeterBowman ?

@PeterBowman
Copy link
Member Author

I think we should keep docker on our radar because of GPU/DNN-ready OpenCV 4.x, but yes, this issue should be dropped as we aim to supersede tensorflowDetection2D. Closing!

@PeterBowman PeterBowman closed this as not planned Won't fix, can't repro, duplicate, stale Jun 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants