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

add a magnetometer topic #25

Closed
wants to merge 2 commits into from
Closed

add a magnetometer topic #25

wants to merge 2 commits into from

Conversation

sameerparekh
Copy link
Contributor

Hi Mani-- I added some code to publish out the magnetometer data on its own topic. I also reformatted a few bits to make it a little easier to read.

@mani-monaj
Copy link
Member

Thank you very much @sameerparekh . I added your proposed changes to the dev-unstable branch. Because of #26 I need to test the new features more before merging them into master. Could you please put some time and test the changes in the dev-unstable branch?

@sameerparekh
Copy link
Contributor Author

I've done a quick test of dev-unstable and it appears to work fine.

-s

On Friday, November 9, 2012 at 9:41 PM, Mani Monajjemi wrote:

Thank you very much @sameerparekh (https://github.com/sameerparekh) . I added your proposed changes to the dev-unstable branch (https://github.com/AutonomyLab/ardrone_autonomy/tree/dev-unstable). Because of #26 (#26) I need to test the new features more before merging them into master. Could you please put some time and test the changes in the dev-unstable branch?


Reply to this email directly or view it on GitHub (#25 (comment)).

@mani-monaj
Copy link
Member

Thank you very much for your feedback.

@mani-monaj
Copy link
Member

FIxed in 0dc6a2a

@mani-monaj mani-monaj closed this Nov 21, 2012
garyservin pushed a commit to garyservin/ardrone_autonomy that referenced this pull request Sep 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants