Skip to content

Pr vision px4 #1

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.

Sign up for GitHub

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

Jump to bottom
Merged
LorenzMeier merged 62 commits into from
May 28, 2018
Merged

Pr vision px4 #1

LorenzMeier merged 62 commits into from
May 28, 2018

Conversation

@RomanBapst
Copy link
Collaborator

@RomanBapst RomanBapst commented May 28, 2018

Same as ATLFlight#12

@LorenzMeier
Copy link
Member

LorenzMeier commented May 28, 2018

Can you rebase? But only if it works out, otherwise please abort.

@RomanBapst
Copy link
Collaborator Author

RomanBapst commented May 28, 2018

@LorenzMeier Ok, let me try

rkintada and others added 27 commits May 28, 2018 09:47
@rkintada @RomanBapst
Updated the VISLAM example for mv_1.0.2 release
553c5df 
Signed-off-by: Ramakrishna Kintada <rkintada@gmail.com>
@rkintada @RomanBapst
Updated the README.md file to add the specifics for mv 1.0.2 release
da026cf 
Signed-off-by: Ramakrishna Kintada <rkintada@gmail.com>
@rkintada @RomanBapst
Minor Formatting update in the README.md file.
1f0d661 
Signed-off-by: Ramakrishna Kintada <rkintada@gmail.com>
@rkintada @RomanBapst
Removed a section that is not needed for mv 1.0.2 release.
58f0b03 
Signed-off-by: Ramakrishna Kintada <rkintada@gmail.com>
@potaito @RomanBapst
Changed package name
05bdc1f
@potaito @RomanBapst
Removed IMUManager dependency completely - still compiling
430fc1a
@potaito @RomanBapst
Temporarily removed tests from Cmake and fixed a missing include
b7701a2
@potaito @RomanBapst
Now using imu ros messages for Vislam - compiling, but untested
a660b07
@potaito @RomanBapst
Added instructions for mavros
4eb6554
@potaito @RomanBapst
Added launch file for mavros + vislam
d4ff9ab
@potaito @RomanBapst
Improved (un-)subscribing to mavros imu topic
627a4a9
@potaito @RomanBapst
Output formatting
eba7ebb
@potaito @RomanBapst
VISLAM working - Converting IMU messages from ENU to NED frame prior …
dcbef95 
…to feeding to VISLAM
@potaito @RomanBapst
Added logging for changes in mv tracking state
2dfc921
@potaito @RomanBapst
Changed mavlink port
21b82fb
@potaito @RomanBapst
Fixed IMU coordinate frame transformation for vislam
0fd9b6f
@potaito @RomanBapst
Renamed project to snapdragon_mavros_vislam
74d5ac8
@potaito @RomanBapst
Fixed launch file
a6cd961
@potaito @RomanBapst
Readme update
94c8f8e
@potaito @RomanBapst
Readme update
946c289
@potaito @RomanBapst
Fixed bug with time stamp conversion
69eb32d
@potaito @RomanBapst
Removed the YZ inversion and corrected the vislam IMU-CAM transformat…
9b02e08 
…ion instead
@potaito @RomanBapst
Added publishers for the estimated camera-imu translation and rotation
85089af
@potaito @RomanBapst
Added PoseWithCovarianceStamped publisher for mavros
70b4242
@potaito @RomanBapst
Comment and formatting
b20a062
@potaito @RomanBapst
Added topic remapping to feed mavros with vislam odometry
db49998
@potaito @RomanBapst
Added px4 configurations files (mainapp.conf and px4.conf)
8ba586d
potaito and others added 26 commits May 28, 2018 09:49
@potaito @RomanBapst
Changelog
9213c41
@potaito @RomanBapst
Launch file edit
6b1954c
@potaito @RomanBapst
Removed unused imu manager
589ded7
@potaito @RomanBapst
Readme update
27823ab
@potaito @RomanBapst
Minor changes in readme etc.
e9084ff
@potaito @RomanBapst
Added mavros tested version info
d8482e4
@potaito @RomanBapst
fix in mainapp.conf
1abcf70
@potaito @RomanBapst
Added offset computation. Now camera and imu timestamps carry the REA…
4d7af95 
…LTIME timestamp, which conforms with ros standards
@ChristophTobler @RomanBapst
Use port 14555 for MavROS and mode magic (no other mavlink streams)
b0541a7
@ChristophTobler @RomanBapst
use mavlink port 14550 for QGC
57de395
@ChristophTobler @RomanBapst
Use imu-cam translation that the algorithm converges to
f1f90f5
@ChristophTobler @RomanBapst
beautify
d5e6cba
@ChristophTobler @RomanBapst
mavros launch: use default px4 mavros config and remap to mavros/odom…
ec0d6a9 
…etry/odom
@ChristophTobler @RomanBapst
remove unused pos_cov msg
ae138ec
@ChristophTobler @RomanBapst
rotate pose to ENU for mavros
cbc375d 
TODO find nicer solution
@ChristophTobler @RomanBapst
update mainapp.conf for ekf2
ffc616e
@ChristophTobler @RomanBapst
remove ekf2 px4.config -> use default PX4/Firmware
6994ff7
@ChristophTobler @RomanBapst
use front-left-up output and add static transforms for mavros
1974893
@ChristophTobler @RomanBapst
replace static transform in code with node in launch file
f0115fe
@ChristophTobler @RomanBapst
update readme
2e3c1ac
@ChristophTobler @RomanBapst
mainapp.config: wait for muorb to start before setting params
af94337
@ChristophTobler @RomanBapst
provide num tracked features instead of bool if updated
f34ab3e
@ChristophTobler @RomanBapst
mark odom_msg invalid by providing large covariance
76ccf03
@ChristophTobler @RomanBapst
update px4 configs
d0955f2 
add mavlink streams that mavros needs (also for avoidance)
@RomanBapst
Divide RAW_IMU by 1000
969ecac 
And some other corrections required for proper snapdragon configuration
@RomanBapst
removed scale on accel, new versions of mavros already take it into a…
9a82a56 
…ccount

Signed-off-by: Roman <bapstroman@gmail.com>
@RomanBapst
Copy link
Collaborator Author

RomanBapst commented May 28, 2018

@LorenzMeier just a minor conflict in the README, the rest applied fine.

@RomanBapst
mainapp.config: replace RAW_IMU with SCALED_IMU which is used in PX4/…
8f3ae0b 
…master

Signed-off-by: Roman <bapstroman@gmail.com>
@LorenzMeier LorenzMeier merged commit 4bf4074 into PX4:master May 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LorenzMeier LorenzMeier LorenzMeier approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@RomanBapst @LorenzMeier @rkintada @potaito @ChristophTobler