RoboticaUtnFrba / create_autonomy

ROS autonomy stack for iRobot Create 2.
BSD 3-Clause "New" or "Revised" License
28 stars 38 forks source link

interfaz proyecto segmentacion #235

Closed lmanfredi closed 3 years ago

lmanfredi commented 3 years ago

Description

Please include a summary of the change and which issue is fixed. Also include relevant motivation and context. If you installed new packages and dependencies make sure they're in the .repos file (vcstools).

Fixes #82

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Also list any relevant details for your test configuration.

# Step 1
./run_command a
# Step 2
./run_another_command

Test Configuration:

Checklist:

eborghi10 commented 3 years ago

@lmanfredi El proyecto tiene una serie de tests para evitar que el código deje de funcionar cuando se agrega nuevo código. Este método se conoce como Integración Continua (CI) y usamos Travis.

Si te fijás acá, falla cuando hace este chequeo que podés repetir localmente:

catkin lint -W2 --strict --explain /path/to/create_autonomy

Y se queja por una par de cosas que son fácilmente solucionables:

WARNING: Package name "ProyectoSegmentacion" does not follow the naming conventions. It should start with a lower case letter and only contain lower case letters, digits, underscores, and dashes.

ProyectoSegmentacion: error: file 'scripts/MoveToGoal_obstacles.py' is executable but not installed

Este último consiste en editar el archivo CMakeLists.txt para instalar los archivos dentro del directorio scripts. Fijate que acá también se hace lo mismo.

eborghi10 commented 3 years ago

Closing in favor of https://github.com/RoboticaUtnFrba/create_autonomy/pull/258