RobotecAI / RobotecAI/ROSCon2023Demo

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.

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

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.

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.

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.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.