Ml-agents: "Decision Frequency" property name is confusing

[danaugrs]

This is different from https://github.com/Unity-Technologies/ml-agents/issues/779.

The higher the "Decision Frequency" the lower the actual frequency at which decisions are made.
The property should be called "Decision Period" instead.

[xiaomaogy]

Hi @danaugrs , this is kind of confusing from your perspective. However 'Decision Period' seems more confusing.

[TashaSkyUp]

Personally decision period makes more sense to me.

[danaugrs]

@xiaomaogy this is confusing from the perspective of anyone who understands what frequency means. The "Decision Frequency" property measures the interval of time between decisions (i.e. the period), not the decision frequency as its name suggests!

If I increase the "Decision Frequency" property value, it will lower the actual decision frequency.
If I lower the "Decision Frequency" property value, it will increase the actual decision frequency.

The period is the reciprocal of the frequency, and that's the quantity measured. The current name is not just confusing - it's plain wrong.

Another idea for the new name is "Decision Interval".

[awjuliani]

Hi all. I agree that `Decision Frequency' is not a good name. We will meet and come up with a new name to use.

[jo3w4rd]

My 2 cents: Decision Interval is better than Decision Period.

Interval means: "an intervening time or space."

Period implies that it takes that long to make a decision, where in fact, the decision still happens in one simulation step no matter what the frequency/period/interval is.

[lock[bot]]

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.