BehaviorParametersEditor OnInspectorGUI constantly recreates sensors

[mbaske]

BehaviorParametersEditor's OnInspectorGUI() calls DisplayFailedModelChecks(), which then calls agent.InitializeSensors(). This causes the CreateSensors() code to be invoked on every OnInspectorGUI callback, even at runtime. The only way to stop repeated sensor instantiation is to hide the BehaviorParameters component inspector, so that its GUI is no longer updated.

To Reproduce
Open the FoodCollector example and place a Debug.Log in GridSensorComponent's CreateSensors() method. Select one of the agent gameobjects in the hierarchy and fold out its behavior parameters inspector settings. Enter play mode.

• ML-Agents version: Release 18
• Environment: FoodCollector

[mbaske]

Here's my workaround for hiding the inspector:

if (Application.isPlaying)
{
    var tracker = UnityEditor.ActiveEditorTracker.sharedTracker;
    var editors = tracker.activeEditors;
    for (int i = 0; i < editors.Length; i++)
    {
        // Can't check type because BehaviorParametersEditor is internal.
        if (editors[i].ToString() == " (Unity.MLAgents.Editor.BehaviorParametersEditor)")
        {
            tracker.SetVisible(i, 0);
        }
    }
}

[ervteng]

Thanks for reporting this bug - i've recorded it in our internal tracker as MLA-2093, and it should be fixed in an upcoming release.

[stale]

This issue has been automatically marked as stale because it has not had activity in the last 28 days. It will be closed in the next 14 days if no further activity occurs. Thank you for your contributions.

[stale]

This issue has been automatically closed because it has not had activity in the last 42 days. If this issue is still valid, please ping a maintainer. Thank you for your contributions.

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.