laser_model_type should not be namespaced

Open Beginner friendly
#324 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
82/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Quiet
Tech stack
yaml
Domain
robotics

Research direction

Start with otto600Params.yaml and otto1500Params.yaml, then search the repository for every YAML definition of laser_model_type. Check each value for the robot namespace and correct the affected configuration entries. Done means all YAML files define laser_model_type without a namespace.

Written by the indexing model from the issue text.

Description

Currently, the 'otto600Params.yaml' and 'otto1500Params.yaml' files contain the following parameters:
laser_model_type': "<robot_namespace>/likelihood_field"
laser_model_type should not be namespaced, and this error should be corrected in all YAML files that define this field.

Dominant language
C++
Stars
98
Forks
22
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from RobotecAI/ROSCon2023Demo

All issues in RobotecAI/ROSCon2023Demo

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.