ml-agents icon indicating copy to clipboard operation
ml-agents copied to clipboard

Reorder angles in RayPerceptionSensor

Open RedTachyon opened this issue 3 years ago • 0 comments

Proposed change(s)

This PR changes the order in which rays are recorded in the RayPerceptionSensor. Currently, they start from the central ray, and then alternate between one to the left and one to the right. This makes it tricky to process these observations with a convolutional layer, which in my experience caused at least some people to reimplement a ray sensor instead of using the built-in one.

Useful links (Github issues, JIRA tickets, ML-Agents forum threads etc.)

Mentioned in Issue #5737

Types of change(s)

  • [ ] Bug fix
  • [x] New feature
  • [ ] Code refactor
  • [x] Breaking change
  • [ ] Documentation update
  • [x] Other (please describe)

Checklist

  • [ ] Added tests that prove my fix is effective or that my feature works
  • [ ] Updated the changelog (if applicable)
  • [ ] Updated the documentation (if applicable)
  • [ ] Updated the migration guide (if applicable)

Other comments

RedTachyon avatar May 15 '22 23:05 RedTachyon