[GH-414] add bush mechanic

[agustinesco]

Closes #414

Now we'll have a new entity type called "Bush" that will work as a non collisionable entity to hide players from other players

The bushes will hide players under the following conditions:

1. Players inside the bush can always see players outside the bush
2. Players outside a bush can't see players inside of a bush
3. If two or more players are inside of the same bush they can only see each other when they're close enough

To tests this pr:

1. add this shape to the bushes array in the config.json file

middle bush

       {
        "position": {
          "x": 0.0,
          "y": 0.0
        },
        "vertices": [
          {
            "x": 0.0,
            "y": 1000.0
          },
          {
            "x": 1000.0,
            "y": 0.0
          },
          {
            "x": 0.0,
            "y": -1000.0
          },
          {
            "x": -1000.0,
            "y": 0.0
          }
        ],
        "radius": 0.0,
        "shape": "polygon"
      }

2. on game_launcher.ex change this: @clients_needed 2
3. on game_updater.ex handle_info(:update_game, %{game_state: game_state} = state) do before the last line add:

Enum.each(game_state.players, fn {player_id, player} ->
  IO.inspect(player.aditional_info.visible_players, label: "visible players for: #{player_id}")
end)

Now start a match in the game client goin to this link and you should see the visible players in the game terminal, when the bot steps on the bush while chasing you, you should disappear from the bot visible players list, something like:

visible players for: 1: []
visible players for: 2: [1]

You can also test this opening two game client windows and play around the bush

[lotuuu]

Functionality looked good by following the steps you gave! Leaving a small code comment

[lotuuu]

I think there's no reason not to use [candidate_player_id | acc] here, right?

It's the better option out of the two, as per ++ docs, and I don't think we care about what order these players are on

[agustinesco]

nope, not reason at all. Changed!

Also i noticed we're doing extra checks since we're running the checks for the same player, i moved the ifs a little bit so we dont do extra calculations

[lotuuu]

Also please add the steps for testing the PR that you sent me via slack in the description!

[agustinesco]

Also please add the steps for testing the PR that you sent me via slack in the description!

yes! sorry i forgot about that, added

[tvillegas98]

Nice work! Although I want to know why instead of implementing something like "Hey I'm on this bush, is there someone else here with me? we implemented something like “Hey, who am I seeing?”