b-it-bots / mas_perception

2 stars 13 forks source link

forward kinetic to latest changes in devel #33

Closed minhnh closed 5 years ago

minhnh commented 5 years ago

As discussed in #32, I and @mhwasil discussed and agreed that now is a good time for merging devel into kinetic. The changes in this PR include:

Additionally, issues #4, #5, as well as documentation of the other packages in the form of README files, may need to be addressed before this merge. On another hand, if the scope of these issues is too big, we can address them in another PR.

minhnh commented 5 years ago

@mhwasil @sthoduka @deebuls @alex-mitrevski please approve so we can merge before German Open (for a clean start).

minhnh commented 5 years ago

For some reason docker push failed at the end of the build. I'm guessing this is an unrelated issue with DockerHub?

argenos commented 5 years ago

Well, there is something fishy going on here. The build fails in travis.com, but not in travis.org, so it's probably something in the configuration now. I also had to re-add the environment variables for Docker Hub which were for some reason not there anymore.

argenos commented 5 years ago

Seems to work now... I fixed a wrongly added environment variable and removed some old service configuration.