forked from KumarRobotics/sloam
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
TODO: delete/cleanup padCloud bs. now debugging trellis
- Loading branch information
mycochang
committed
Apr 16, 2024
1 parent
05fb260
commit 4d34452
Showing
7 changed files
with
204 additions
and
173 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
We should have noticed earlier that the FOV parameter is half of the real one.
In the ouster config it is set to
22.5 deg
which is half of45 deg
. Ultimately, this is the result of a lack of documentation from the original authors, but this is what you get from open source code written for a paper.BTW shouldn't this be

15.5
? The FOV of the XT32 is31
, although the specs are confusing:If the FOV is not symmetrical in HESAI, then we might need to add a second parameter for top and bottom FOV.