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

Communication strength with breadcrumbs #488

Closed
dsquez opened this issue Jul 16, 2020 · 9 comments
Closed

Communication strength with breadcrumbs #488

dsquez opened this issue Jul 16, 2020 · 9 comments
Assignees
Labels
bug Something isn't working critical

Comments

@dsquez
Copy link
Contributor

dsquez commented Jul 16, 2020

We have started experimenting with the breadcrumb functionality, and we are surprised by the apparent model for determining communication strength. New breadcrumbs appear to expand signal along line of sight (which makes sense), but this seems to be limited to within a certain radius of the staging area (or wherever the reference vehicle is when using the instructions provided in the wiki), which isn't what we would expect when setting up a communication system. I have created a google slides presentation that outlines where we put breadcrumbs and the subsequent changes in signal on Simple Cave 2. Does this follow the intended behavior, or might there be a bug?

https://docs.google.com/presentation/d/1tyE6HdsrkdCDjlSVKqXAX9lrhFQW6nzWBoz35OolC9Q/edit?usp=sharing

@caguero
Copy link
Contributor

caguero commented Jul 31, 2020

Hi @dsquez , thanks for the great report. We're right now trying to reproduce your results. We'll go back to you soon.

@caguero
Copy link
Contributor

caguero commented Aug 3, 2020

@dsquez , I can confirm that we found an issue with breadcrumbs. We're working on the fix.

@malcolmst
Copy link

Just curious if there is any update on this issue? I haven't been experimenting much with the breadcrumbs yet but I see the PR for the fix is still open

@nkoenig
Copy link
Contributor

nkoenig commented Sep 11, 2020

The PR is still open due to qualification code-freeze. We will be integrating communication updates next week.

@malcolmst
Copy link

Makes sense, thanks for the update!

@DalInar
Copy link

DalInar commented Sep 25, 2020

Hello! I've been experimenting with the breadcrumbs now that the updated comms model has been merged into master (#579, also related to #587 ), and it's behaving a bit differently than I (perhaps naively) expected. Based on the new changes to the comms model, could you answer the following question:

What is the intended behavior of the RSSI value returned by the getNeighbors function when breadcrumbs are deployed? In particular, say I have a TEAMBASE stationed in the starting area. Should I expect the RSSI between a robot and TEAMBASE to suddenly improve when a breadcrumb is deployed, because the RSSI is based on the connection between the breadcrumb and the robot, or should I simply expect the RSSI to fall off less quickly as I move away from the breadcrumb because the RSSI is somehow based on the entire path back to the starting area?

My simple experiments today (just manually moving a robot into the tunnel, manually deploying breadcrumbs, and then monitoring the RSSI returned by getNeighbors) seems to imply that the answer is the latter, but I was hoping for the former :-)

@malcolmst
Copy link

malcolmst commented Sep 30, 2020

Agree that RSSI to the nearest breadcrumb would be nice information to have (would likely be the only RSSI information easily available in the real world, or at least it would be available in addition to the “longest path” RSSI)

@acschang
Copy link
Contributor

Documentation regarding the revisions to the communication model has been updated on the wiki.

An example of how breadcrumbs function under similar conditions to those originally reported are shown below:
ssci_sw02

To clarify for @DalInar: the currently reported RSSI values are based upon the entire path to the noted end-point. For more discussion around RSSI values please refer to #643.

@caguero
Copy link
Contributor

caguero commented Jan 18, 2021

I believe the original issue reported by @dsquez has been addressed. I'm closing this issue but follow up on #643 for RSSI problems.

@caguero caguero closed this as completed Jan 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working critical
Projects
None yet
Development

No branches or pull requests

6 participants