RaycastPerceptionComponent custom editor

[chriselion]

Proposed change(s)

• Make RayPerceptionSensorComponent fields private. Most of them now have public getters and setters. Ones that affect the observation shape of the sensor (num rays, tags, and stacks) properties are not public. This allows changing of some sensor values at runtime (e.g. someone asked for changing the ray length as part of generalization).
• Add a custom editor for RayPerceptionSensorComponent. Parameters that aren't allowed to change during training/inference get disabled in play mode.
• Make debug drawing for RayPerceptionSensorComponent work outside of play mode. Slight behavior change in that only selected Agent(s) are drawn.

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

https://jira.unity3d.com/browse/MLA-688
https://jira.unity3d.com/browse/MLA-561

#3412
https://forum.unity.com/threads/no-visible-gizmos-for-ray-perception-sensor-component-3d.824529/
https://forum.unity.com/threads/access-rayperceptionsensor-components.823107/

Types of change(s)

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

Checklist

• I have added tests that prove my fix is effective or that my feature works
• I have added updated the changelog (if applicable)
• I have added necessary documentation (if applicable)
• I have updated the migration guide (if applicable)

Other comments

[chriselion]

[surfnerd]

nit: attributes should go on the same line if they fit.

[surfnerd]

nit: remove

[chriselion]

Will do. I'm not quite clear why we have this in the first place for things like this:

ml-agents/com.unity.ml-agents/Runtime/Policy/BehaviorParameters.cs

Lines 35 to 37 in 9450d3f

	[HideInInspector]
	[SerializeField]
	string m_BehaviorName = "My Behavior";

but then expose it in the editor:

ml-agents/com.unity.ml-agents/Editor/BehaviorParametersEditor.cs

Line 28 in 9450d3f

EditorGUILayout.PropertyField(so.FindProperty("m_BehaviorName"));

I had to remove it here to get the list to expand properly.

[surfnerd]

LGTM.

Food for thought: I wonder if these would be helpful in a player build as well. 🤔

[chriselion]

I wonder if these would be helpful in a player build as well.

Do you mean the gizmo drawing? What's the typical setup?