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

Fixed WF agent to work more reliably. #257

Merged
merged 4 commits into from
Apr 14, 2022
Merged

Fixed WF agent to work more reliably. #257

merged 4 commits into from
Apr 14, 2022

Conversation

laurajogi
Copy link
Collaborator

Agent checks if wall is followed before when it has found a new wall while doing heuristics.
Basically uses a pheromone/marker idea to check if walls are followed before, but the pheromone smell (or anything like that) has not yet been implemented.

@laurajogi laurajogi linked an issue Apr 13, 2022 that may be closed by this pull request
@laurajogi laurajogi marked this pull request as ready for review April 13, 2022 13:30
S010MON
S010MON previously approved these changes Apr 14, 2022
Copy link
Owner

@S010MON S010MON left a comment

Choose a reason for hiding this comment

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

Looks really good, one slight bug, which is that the agents don't seem to go through the portals. Definitely ready to push, perhaps deal with that issue later.

@laurajogi
Copy link
Collaborator Author

Tested the WF agent on a simple map with many portals and they do go through the portals when they go directly towards it, i.e. the portal is in front of the agent. But when it's just to the left or right of them, they don't recognise it as a corner to turn around at and they go past 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.

Great work.

@S010MON S010MON merged commit 961a1a9 into master Apr 14, 2022
@S010MON S010MON deleted the WF-reliability-fix branch April 14, 2022 18:29
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.

WF agent check if wall is already explored before following
3 participants