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

Gabe noisy light #40

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Gabe noisy light #40

wants to merge 2 commits into from

Conversation

SeymourGabe
Copy link
Collaborator

@SeymourGabe SeymourGabe commented Jul 25, 2019

For issue #19: Took 50 measurements of 3 different colors (white, grey, black) on 15 different real Kilobots. Then analyzed the measurements to find the distribution of each of these colors. I found that the standard deviation of the light measurements depends on the intensity of the light being measured (more light->higher standard deviation). I then fit a second-degree polynomial to the std_dev of the measurements I took. Each time get_ambientLight() is called, the standard deviation polynomial uses the noise-free measurement to find the std_dev at that specific measurement. A normal distribution random number generator finds the measurement with noise.

… different bots. I then analyzed this to find the distribution of each of these colors. I found that the standard deviation of the light measurements depended on the intensity of the light being measured (more light->higher standard deviation). Fit a second degree polynomial to the measurements I took. Thus, each time get_ambientlight() is called, the standard deviation polynomial uses the noise-free measurement to find the std_dev at that measurement and a normal distribution random number generator finds the measurement with noise.
@jtebert
Copy link
Owner

jtebert commented Jul 25, 2019

That's so overkill. I love it.

I'm going to wait to merge this until we figure out the multi-threading issue, in case the problem is related to the threaded random number generation.

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.

None yet

2 participants