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

Added sparse mapping launch script #374

Merged
merged 1 commit into from
Dec 23, 2021

Conversation

rsoussan
Copy link
Member

Runs the sparse mapping matcher (localization_node) live using an input bagfile and optionally saves the results to a bagfile. Created for @oleg-alexandrov for use with the ISAAC pipeline.

Copy link
Member

@bcoltin bcoltin left a comment

Choose a reason for hiding this comment

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

Maybe pass the terminal as an argument to the launch file and default to a more standard one than terminator. Otherwise looks good.

@oleg-alexandrov oleg-alexandrov merged commit 4af88c0 into nasa:develop Dec 23, 2021
@oleg-alexandrov
Copy link
Contributor

oleg-alexandrov commented Dec 23, 2021

I pulled this in, I tested that it works, except for the last line,

node pkg="rosservice" type="rosservice" name="enable_sparse_mapping" args="call --wait /loc/ml/enable true"

failing. I wrote an email about that.

oleg-alexandrov added a commit that referenced this pull request Dec 23, 2021
@oleg-alexandrov
Copy link
Contributor

I think I am very bad at dealing with pull requests. I merged this in prematurely, the clicked on "revert" to see what it will do, which just created a branch, which I plan to wipe.

I think there's not much harm I pulled this in too early. I will keep an eye on the unresolved issue above which shows up only on ROS melodic and not with ROS kinetic.

@bcoltin
Copy link
Member

bcoltin commented Dec 24, 2021

It's fine, just make a new pull request when you fix that problem.

@oleg-alexandrov
Copy link
Contributor

I will look at this this week.

luisa-mao pushed a commit to luisa-mao/astrobee-1 that referenced this pull request Jul 11, 2023
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.

3 participants