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

Simple Tutorial RViz launch missing -d path to .rviz file #357

Closed
cod3monk3y opened this issue Jul 18, 2023 · 2 comments · Fixed by #395
Closed

Simple Tutorial RViz launch missing -d path to .rviz file #357

cod3monk3y opened this issue Jul 18, 2023 · 2 comments · Fixed by #395
Labels
enhancement New feature or request

Comments

@cod3monk3y
Copy link

cod3monk3y commented Jul 18, 2023

Under the ROS 1 section for the Simple Tutorial, the launch command reads

rviz # term 2

Executing that directly will show Fixed Frame [map] does not exist because RViz is using default.rviz configuration. I had to specify the .rviz file using -d to get the visualization working:

$ cd ~/openvs_ws
$ rviz -d src/open_vins/ov_msckf/launch/display.rviz

After this it works great!

Is this just my setup, or should the doc change from just rviz to something like: rviz -d src/open_vins/ov_msckf/launch/display.rviz?

@cod3monk3y
Copy link
Author

cod3monk3y commented Jul 19, 2023

I assume yes, since I see now that you do this in the demo video.

@goldbattle goldbattle added the enhancement New feature or request label Jul 19, 2023
@goldbattle
Copy link
Member

There is a comment above that about what config to open

For RVIZ, one can open the ov_msckf/launch/display.rviz configuration file.

But directly from command line might help those newer to the project

goldbattle added a commit that referenced this issue Oct 29, 2023
@goldbattle goldbattle mentioned this issue Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants