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

Update map.yaml #384

Closed
wants to merge 19 commits into from
Closed

Update map.yaml #384

wants to merge 19 commits into from

Conversation

alsora
Copy link

@alsora alsora commented Feb 4, 2019

update map.pgm relative path

Without this change, the map server crashes with the following error

[ERROR] [map_server]: failed to open image file "/root/turtlebot3_ws/install/turtlebot3_navigation2/share/turtlebot3_navigation2/map/~/map.pgm": Couldn't open /root/turtlebot3_ws/install/turtlebot3_navigation2/share/turtlebot3_navigation2/map/~/map.pgm

oiz5201618 and others added 19 commits July 31, 2018 18:00
The correct parameter name in dwa_local_planner is "vth_samples".
Added an additional argument move_forward_only to prohibit backward l…
Enable turtlebot3_teleop_key to work on Windows
The math.ceil() output in some cases is different in python2 and python3, for this reason I change the operation by the standard floor operation
change math.ceil() operation
Signed-off-by: Pyo <pyo@robotis.com>
updated the CHANGELOG and version to release binary packages
update map.pgm relative path
@routiful routiful changed the base branch from master to ros2 February 7, 2019 01:53
@routiful
Copy link

routiful commented Feb 7, 2019

Thank you for your suggestion. But it is get some conflict to merge from your branch.
Please check your commits on your branch.

@alsora
Copy link
Author

alsora commented Feb 7, 2019

I'm sorry, for some reason I was trying to merge from the wrong branch.
I reopened the correct PR #386

@alsora alsora closed this Feb 7, 2019
@kaym9n kaym9n removed the in progress label Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants