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

Aco agent limited vision #232

Merged
merged 22 commits into from
Mar 18, 2022
Merged

Aco agent limited vision #232

merged 22 commits into from
Mar 18, 2022

Conversation

trav-d13
Copy link
Collaborator

A.C.O Agent with limited visual field completed. Fulfills criteria of either moving/ looking every n seconds.
Additionally utilizes a short-term memory to handle glass/ agent clashes/ ect.
Addition of heuristics to make better!
Closes #231

@trav-d13 trav-d13 self-assigned this Mar 17, 2022
@S010MON
Copy link
Owner

S010MON commented Mar 17, 2022

This is failing because of the requirements of the WF agent

@laurajogi
Copy link
Collaborator

Do we have to have the limited view? Because even in the project manual/FAQ they said the agent can see in front, and directly left and right, so why limit it? Otherwise, if we really want to have the option, I can work on the fix for WF agent tomorrow.

@S010MON
Copy link
Owner

S010MON commented Mar 18, 2022

Do we have to have the limited view? Because even in the project manual/FAQ they said the agent can see in front, and directly left and right, so why limit it? Otherwise, if we really want to have the option, I can work on the fix for WF agent tomorrow.

@laurajogi - front, left and right is a limited view: there is no back and therefore no 360

…co_agent_limited_vision

# Conflicts:
#	src/main/java/app/model/agents/AgentImp.java
S010MON
S010MON previously approved these changes Mar 18, 2022
src/main/java/app/model/agents/ACO/AcoAgent360Vision.java Outdated Show resolved Hide resolved
src/main/java/app/model/agents/Grid.java Show resolved Hide resolved
src/main/java/app/model/agents/Grid.java Show resolved Hide resolved
@S010MON
Copy link
Owner

S010MON commented Mar 18, 2022

Approved with some suggestions for change

Copy link
Collaborator

@laurajogi laurajogi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works fine for me. I will try to solve the issue for WF agent as well soon, but first need to take care of some other movement issues for it.

Copy link
Collaborator

@MatthewFoster02 MatthewFoster02 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work, ready to merge.

@trav-d13 trav-d13 merged commit b782ce7 into master Mar 18, 2022
@trav-d13 trav-d13 deleted the Aco_agent_limited_vision branch March 18, 2022 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Aco Agent with Limited visual field
5 participants