[DISCOVERY] Duplicate value in observation function

[Giovannibriglia]

Issue:

• In the discovery scenario, I noticed that in the observation function, at this line

  VectorizedMultiAgentSimulator/vmas/scenarios/discovery.py

  Line 236 in ee892d8

  agent.state.pos,

  , the agent.state.pos is included twice in the observation array.

Question:

• What is the intended purpose of including agent.state.pos twice in the observation? Is this behavior correct?

Suggestion:

• Could the comment part at this line

  VectorizedMultiAgentSimulator/vmas/scenarios/discovery.py

  Line 238 in ee892d8

  # lidar_2_measures,

  be used to replace the duplicate?

Thank you :)

[matteobettini]

Will remove the duplicate. Lidar 2 was removed as we don't allow to see other agents (need to communicate) but will put a kward to have it back