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

Coverage map #237

Merged
merged 11 commits into from
Mar 20, 2022
Merged

Coverage map #237

merged 11 commits into from
Mar 20, 2022

Conversation

S010MON
Copy link
Owner

@S010MON S010MON commented Mar 17, 2022

Creates a discrete coverage map of the continuous map with a set sensitivity and pixel size that will allow for comparison of sets of vectors and produce percentage accuracy of how many walls have been observed on the map.

@S010MON S010MON marked this pull request as draft March 17, 2022 19:36
@S010MON S010MON requested a review from MatthewFoster02 March 17, 2022 19:36
@trav-d13
Copy link
Collaborator

Hey mate, just ran the build my side, the build completes, however there are two failing tests. he stack track is below:
Task :test

CoverageTest > testCoverageMapWithFurniture() FAILED
java.lang.ArrayIndexOutOfBoundsException at CoverageTest.java:46

CoverageTest > testCoverageMapEmpty() FAILED
java.lang.ArrayIndexOutOfBoundsException at CoverageTest.java:18

@S010MON S010MON marked this pull request as ready for review March 20, 2022 08:23
S010MON added 2 commits March 20, 2022 09:25
# Conflicts:
#	src/main/java/app/model/agents/Agent.java
#	src/main/java/app/model/agents/AgentImp.java
@S010MON S010MON requested a review from trav-d13 March 20, 2022 08:30
@S010MON S010MON merged commit a2777f4 into master Mar 20, 2022
@S010MON S010MON deleted the coverage-map branch March 20, 2022 09:46
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.

3 participants