Reworked and Stable Sonar Object/Wall Detection#109
Conversation
…data. Docker-build.sh changed because of issues with personal computer.
Added Sonar Denoiser Class which can process sonar sweeps. Added a publisher to allow viewing data and testing if the processing is actually working
…ry code for the test
… with the sonar device to sonar_utils.py (#97) - Skipping checks because this is just to merge two of my branches together so that I can work out of one.
…stances of objects
…ressed_img, code in task_planning is from Pranav Bijith - add rotate_to_normal and rotate_to_angle_from_normal
…onstant_sweep method align with the new sonar pipeline
|
Converting to draft as has not been pool-tested yet. |
|
Only sonar tasks need pool testing now. |
This reverts commit bec9f5b.
…s2 into sonar-processing
…s2 into sonar-processing Update from oogway code
…s2 into sonar-processing
Merging main into sonar-processing to prepare for PR.
|
I reopened the PR after thorough testing in the pool. |
nathanaelren
left a comment
There was a problem hiding this comment.
i'll let @ChuChuCodes0414 fully review this (I'm in Greece) but took a quick look since my flight is delayed, two quick comments, thanks for all the great work Anthony! This is awesome
There was a problem hiding this comment.
fix floating point errors
|
|
||
| ## Config | ||
| The `config` directory contains robot-specific `.yaml` files. The format is as follows: | ||
| ```yaml |
There was a problem hiding this comment.
data type for all these pls
ChuChuCodes0414
left a comment
There was a problem hiding this comment.
In general good PR. Some changes to be made.
I will take a closer look when these are reviewed along with Nathanael's comments. Thank you!
|
|
||
| # Normalize the angle to be within a specific range (e.g., -180 to 180) | ||
| line_angle_deg = np.clip(line_angle_deg, -180, 180) | ||
| segmentation = sonar_object_detection.SonarSegmentation2( |
There was a problem hiding this comment.
Can SonarSegmentation2 have a more descriptive name to differentiate it from SonarSegmentation
|
|
||
| return nearest_segment | ||
|
|
||
| class SonarSegmentation2: |
There was a problem hiding this comment.
Same as previous comment
|
|
||
| # Move back through gate | ||
| await move_with_directions([(-2.5, 0, 0)], depth_level=DEPTH_LEVEL) | ||
| await move_with_directions([(2.5, 0, 0)]) |
There was a problem hiding this comment.
No need for these prequal changes here
There was a problem hiding this comment.
Let's also not push this to main please.
Summary
Testing