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

Cleanup EKF #11

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Cleanup EKF #11

wants to merge 4 commits into from

Conversation

kevin-robb
Copy link
Member

Should function the same as it did at competition, just more compartmentalized, simplified, and with dead code removed.

… details.

Compiles but cannot test performance with simulator as it only receives messages for IMU but nothing else.
…s of the EKF.

All of Justin's EKF code can be accessed in the IGVC 2020 repository, so it was removed here.
@kevin-robb
Copy link
Member Author

Having trouble testing this using rosrun/roslaunch. It seems like only the IMU is being published from the simulator, and not the GPS or velocities. Noah, can you try running it to see if this is the case? I've verified the topic names match, and am no longer receiving any syntax errors. I'm not sure if the way I set up the code is causing it to have problems or if it's not getting data from the sim as I suggested.

@kevin-robb
Copy link
Member Author

I have this suspicion because I put a print statement in every subscriber's function, and only the IMU is printed to the console. The EKF state is only changing in yaw/yawrate, and not x/y or v_l/v_r.

…can be modified in only the ROS node for testing.
Various tweaks to attempt to get the EKF to work, mainly in the update() function.
Also change measure_velocities to assume it's receiving angular.
EKF is performing without errors but poorly.
@kevin-robb
Copy link
Member Author

Having trouble testing this using rosrun/roslaunch. It seems like only the IMU is being published from the simulator, and not the GPS or velocities. Noah, can you try running it to see if this is the case? I've verified the topic names match, and am no longer receiving any syntax errors. I'm not sure if the way I set up the code is causing it to have problems or if it's not getting data from the sim as I suggested.

This issue was my bad, only sourcing the workspace when running the ROS node, but not when running rosbridge. I added a launch file to solve this.

At present the EKF is simply not performing very well in the simulator, so I will not bring this version into master. I may come back and fix it but who knows.

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.

None yet

1 participant