Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rework getClosestMine to check across the edge of the screen #1

Open
se-ke opened this issue Feb 18, 2019 · 0 comments
Open

Rework getClosestMine to check across the edge of the screen #1

se-ke opened this issue Feb 18, 2019 · 0 comments
Labels
bug Something isn't working

Comments

@se-ke
Copy link
Owner

se-ke commented Feb 18, 2019

Currently, getClosestMine works by checking the distance between each agent and each mine as if they were on a flat surface, a rectangle. However, the edges of the rectangle are in fact linked together to form a torus. So, if an agent is on the left side of the screen, it will not determine that a mine on the right side of the screen is closest to it because it does not account for the edges being connected. This could be fixed by testing multiple location of the mine (ex. (x,y), (x-width, y), (x, y-height), (x-width, y-height), (x+width, y), (x, y+height), (x+width,y+height)), but this is rather burdensome.

@se-ke se-ke added the bug Something isn't working label Nov 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant