Skip to content

Read the tof-sensors in parallel #224

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

Merged
merged 19 commits into from
Jul 5, 2021
Merged

Read the tof-sensors in parallel #224

merged 19 commits into from
Jul 5, 2021

Conversation

amandel
Copy link
Member

@amandel amandel commented Feb 17, 2021

  • primary sensor gives pace
  • no wait for result, just get what is ready
  • max 30 readings per minute in csv
  • nerd display shows number of echo triggers of primary sensor in the middle
  • do not report timeout readings (should we do this?)
  • inner loop runs more often (no wait())
  • need a delay(1)?
  • SENSOR_QUIET_PERIOD_AFTER_START_MICRO_SEC to 30ms (was 35)
  • relates Switch back to parallel tof-sensor reading #221

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 6 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

- primary sensor gives pace
- no wait for result, just get what is ready
- max 30 readings per minute in csv
- nerd display shows number of echo triggers of primary sensor in the middle
- do not report timeout readings (should we do this?)
- inner loop runs more often (no wait())
- need a delay(1)?
- SENSOR_QUIET_PERIOD_AFTER_START_MICRO_SEC to 30ms (was 35)
- relates #221
- log measurement loop speed
- call display only every 20ms max.
@amandel amandel force-pushed the feature/parallel-tof branch from b9c8ffb to ff31ab7 Compare May 2, 2021 09:45
@boldt
Copy link
Contributor

boldt commented Jul 3, 2021

Sind wir uns sicher, dass wir das wollen? Ich hatte das vor einem Jahr auch mal eingebaut um die Anzahl der Messungen zu erhöhen. Damals gab's aber Messfehler, weil z.B. der linke Sensor das Signal des rechten empfangen hat. Wir hatten das dann wieder Rückgängig gemacht.

@tobst weißt Du noch mehr Details?

@amandel
Copy link
Member Author

amandel commented Jul 4, 2021

Wir hatten das auf abwechselnd umgestellt, um auszuschließen, dass die TOF-Sensor-Boards sich gegenseitig stören. Das war insbesondere im Zusammenhang mit der Abschirmung. Es ist aber so, dass es bei den stabilen Sensoren zu keiner Beeinflussung kommt.
Um eine Überlappung zu vermeiden müssten wir eine weitere Verzögerung einbauen, so was wie: Wenn von gegenüberliegenden Sensor eine Messung vorliegt, dann warte mindestens Xms seit Start der gegenüberliegenden Messung. Aktuell ist X unabhängig ob eine Messung erfolgte (also ein Echo empfangen wurde) 30ms in der Zeit hat der Schall knapp 3,5 zurückgelegt.
Die Empfindlichkeit ist laut Spec so etwa bei 8m Laufzeit (4m Entfernungsmessung). Das reflektieren dämpft sicherlich deutlich, wir sehen auch bei Tests gegen Wänden dass eher schon unter idealen Bedingungen bei 2,5-3m Schluss ist. Wenn wir davon ausgehen, dass ein 2x reflektierter Ton maximal nach 6m noch zu erfassen ist, müsste X 51.7ms sein.

Meinungen?

Edit, da ich erst nach > TIMEOUT ms feststellen kann ob von dem gegenüberliegenden Sensor eine Messung vorliegt und TIMEOUT bei 75ms liegt braucht es die Fallunterscheidung nicht. Es würde also immer frühestens X ms nach Start der gegenüberliegenden Messung eine neue Messung gestartet.

Edit2: Das führt allerdings auch dazu, dass rechts nicht gemessen wird wenn der Messzyklus des linken Sensors regelmäßig in unter 50ms durch ist. Das sollte OK sein - der hat ja prio.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 4, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@amandel amandel marked this pull request as ready for review July 4, 2021 13:26
@amandel
Copy link
Member Author

amandel commented Jul 5, 2021

Talkes in monday meeting about this - will merge and create a pre-releaae

@amandel amandel merged commit 1086d49 into master Jul 5, 2021
@amandel amandel deleted the feature/parallel-tof branch August 5, 2021 19:19
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.

2 participants