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

Debug configs added for digiforest bags #1

Open
wants to merge 37 commits into
base: NetInput-check
Choose a base branch
from
Open

Conversation

mycochang
Copy link

TODO squash mess later.

added xterm gdb args to sloam/digiforest .launch files

more edits accessing PRBonn/lidar-bonnetal RangeNet++ directories from
segmentation.

re: model:


"just keep it in the ~/Downloads/ folder"


B)

added comments for personal clarity/ease of reading.

mycochang added 3 commits March 12, 2024 14:34
also created some rgs, xps, yps private vars for `use_hesai: true`
debugging. can probably be removed, bc using `lidar_undistortion` from
ori-drs.
added gdb statements to sloam/digiforest `.launch` files

more edits accessing PRBonn/lidar-bonnetal RangeNet++ directories from
segmentation.

re: model:
*******************************************
"just keep it in the `~/Downloads/` folder"
*******************************************
B)

added comments for personal clarity/ease of reading.
@mycochang mycochang requested a review from mcamurri March 12, 2024 14:16
mycochang and others added 26 commits March 14, 2024 16:22
still requires:
- [ ] modified hesai_undistortion.launch file to receive param args from
external launch.
- [ ] sloam_digiforest.yaml params to change to
/hesai/pandar_undistorted topic.
tf debugging still in progress...
also cut rgs arg out of _doProjection() call
still debugging tempCloud OG vs. temp2Cloud NEW

-[x] not the `mask.at<uchar>`
-[ ] trying to pre-populate the `org_pc` w `NaN`s
-[ ] and then filter it.
still need to fix .pcd save functions.
saves it all in the Downloads folder.
-[ ] un hard-code paths
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