Skip to content

[stereocam] uartrotation->state2camera, streamline naming stereo_cam,… #1433

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

Merged
merged 5 commits into from
Nov 23, 2015

Conversation

dewagter
Copy link
Member

  • all related files in stereocam directory
  • uartrotation->state2camera
  • stereo_cam -> stereocam
  • stereo -> stereocam
  • stereoavoid = the avoid version that plans around obstacles during a nav_line auto2 flight -> nav_line_avoid
  • avoid with random exploration -> stereocam_droplet

@dewagter
Copy link
Member Author

  • build server checks
  • inform involved users

break;
}

#if STEREO_LED
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be #ifdef

@flixr
Copy link
Member

flixr commented Nov 21, 2015

Like the cleanup, thx!

@flixr
Copy link
Member

flixr commented Nov 21, 2015

For what build server checks are you waiting there?
All confs are only tested nightly on the master branch....

@dewagter
Copy link
Member Author

@rmeertens
@kirkscheper

Please note your modules were renamed. All stereocam modules are in the stereocam module folder and module names are grouped with the stereocam_ prefix.

All on-autopilot-board (linux-only) computer vision modules are grouped with prefix cv_

@rmeertens
Copy link
Contributor

Thanks for the warning. Love the changes!

dewagter added a commit that referenced this pull request Nov 23, 2015
[stereocam] uartrotation->state2camera, streamline naming stereo_cam,…
@dewagter dewagter merged commit 4edc154 into master Nov 23, 2015
@dewagter dewagter deleted the cleanup_stereo branch November 23, 2015 10:33
@flixr flixr added the Module label Nov 23, 2015
@flixr flixr added this to the v5.8 milestone Nov 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants