Mainly hololens visualization changes + uncommited changes before MSV2018

robofit / arcor

Augmented reality-based human-robot interaction.

GNU Lesser General Public License v2.1

8 stars 14 forks source link

Mainly hololens visualization changes + uncommited changes before MSV2018 #152

Closed xBambusekD closed 5 years ago

xBambusekD commented 5 years ago

PR Checklist:

[ ] Is it tested in lab?
[ ] Are there unit tests?
[ ] Is there (updated) README?

ZdenekM commented 5 years ago

@xBambusekD Jsou tam nějaké konflikty - musíš pullnout aktuální kód z robofit/master, pak vyřešit konflikty (git mergetool), commitnout a pushnout k sobě (nový commit se automaticky přidá do tohodle PR).

ZdenekM commented 5 years ago

@xBambusekD Fail viz https://travis-ci.org/robofit/artable/builds/454091176#L4715:

art_projected_gui/src/art_projected_gui/gui/ui_core_ros.py:960:9: E265 block comment should start with '# '

...to by mělo být snad všechno. Jinak před pushem/PR je dobré si pustit testy lokálně (run_tests.sh). Na CodeFactor se vždycky mrknu (občas tam bývají dobré tipy), ale beru to jen jako doporučení. Na Travisu to musí projít...

ZdenekM commented 5 years ago

@xBambusekD Dík!