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.

Sign up for GitHub

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

Jump to bottom

In example scenario file sentry tower has inside area #2

Open
Isinlor opened this issue Feb 27, 2020 · 0 comments
Open

In example scenario file sentry tower has inside area #2

Isinlor opened this issue Feb 27, 2020 · 0 comments
Labels
guidlines issue An issue with guidelines

Comments

@Isinlor
Copy link
Contributor

Isinlor commented Feb 27, 2020 •
edited
Loading

In the guidelines there is no definition what inside area in scenario definition of sentry tower "ix1,iy1,ix2,iy2 (i = inside area)" means. I think it was based on an old idea of how sentry tower should work.

Current idea defined in the guidelines is:

Sentry Towers
- A sentry tower can be entered from every side.
- An agent in a sentry tower can see over walls.
- An agent in a sentry tower has an increased long range view but cannot see in shorter distances.

viewRangeSentry = 2.0, 20.0 (not visible short range, visible high range)

See also the red circles that were supposed to represent a view from the sentry tower:
image
@Isinlor Isinlor added the guidlines issue An issue with guidelines label Feb 27, 2020
jan-krueger referenced this issue in jan-krueger/GameInterop Mar 15, 2020
@jan-krueger
fixed getObjectPerceptsForAgent
d4a8bb3 
Okay, so the issue was that (1) we didn't provide the coordinates relative to the agent with respect to rotation and position, and (2) sometimes the FieldOfView isInRange implementations calculation different at the 10+th decimal place and therefore disagreed with our collision calculations. I fixed this by simply adding it as a second filter; might not be pretty and efficient but it works. 
fixed getObjectPerceptsForAgent #2


fixed getObjectPerceptsForAgent #2
Isinlor pushed a commit that referenced this issue Apr 15, 2020
@jan-krueger
Merge pull request #2 from waterfl0w/ui-intgration
e0c5aab 
Ui intgration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
guidlines issue An issue with guidelines
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Isinlor