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.

Sign up for GitHub

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

Jump to bottom

fix/cuda pointer data #24

Open
wants to merge 62 commits into
base: main
Choose a base branch
from
Open

fix/cuda pointer data #24

wants to merge 62 commits into from
+469 −75

Conversation

danielsanchezaran
Copy link
Collaborator

This PR adds several (possible?) fixes:

  1. Sets num_targets to Target NPC + 1 (for ego)
  2. Fixes the pre-process calculation, so the yaw used for calculating sine and cosine is the transformed yaw
  3. Adds a small utils library to help print out data streams (for debugging only)
  4. Sets cuda pointers to point to 0.0s and not random values (?)

ktro2828
ktro2828 requested changes Dec 24, 2024
View reviewed changes
lib/src/preprocess/agent_preprocess_kernel.cu Outdated Show resolved Hide resolved
include/autoware/mtr/utils.hpp Outdated Show resolved Hide resolved
include/autoware/mtr/agent.hpp Outdated Show resolved Hide resolved
include/autoware/mtr/utils.hpp Outdated Show resolved Hide resolved
@danielsanchezaran danielsanchezaran marked this pull request as draft December 25, 2024 01:01
@danielsanchezaran danielsanchezaran marked this pull request as ready for review December 26, 2024 00:31
danielsanchezaran and others added 24 commits January 17, 2025 11:19
@danielsanchezaran
solve issue with target number
ed653ae 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
add print utils for debugging
0d16e48 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
fix yaw calculation in pre-processing
c6d520f 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
set memory to point to 0 values and not garbage
3eafafb 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
Revert "fix yaw calculation in pre-processing"
4dfda91 
This reverts commit 8784fcf.
@danielsanchezaran @ktro2828
Update include/autoware/mtr/utils.hpp
986df76 
Use safer std::vector

Co-authored-by: Kotaro Uetake <60615504+ktro2828@users.noreply.github.com>
@danielsanchezaran @ktro2828
Update include/autoware/mtr/utils.hpp
7a63b35 
use a static array instead of a vector

Co-authored-by: Kotaro Uetake <60615504+ktro2828@users.noreply.github.com>
@danielsanchezaran
Revert "solve issue with target number"
8062bf9 
This reverts commit bf37af3.
@danielsanchezaran
fix mismatch between ptr and vector
f56303b 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
add ego to targets
6541200 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
set targets to 2
58247f5 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
set strict num targets
0d845c2 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
resolve conflict
6c903ef 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
add defaults to queue
b42efc0 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
define constructors
ca5225e 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
define constructors not default
734c3c2 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
initialize
f41e8a1 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
use aptr to initialize the queue with the constructor
716d35f 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
Revert "use aptr to initialize the queue with the constructor"
d0bcad5 
This reverts commit aa37b88.
@danielsanchezaran
wip fix queue
449e0c8 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
wip fix queue
286b252 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
wip fix history copying
8d75adb 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
wip fix is valid latest
d0afd8c 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
add prints to help debug pre-process
315c3e1 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
fix back center
b08a53a 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
add polyline label id info
e774373 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
print output
41b8579 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
fix brackets
97647fc 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
fix variable
e0e7062 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
f
04d327a 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
fix
73c51cc 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
handle nan
6c95924 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
prints
baf41f2 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
add label to fromPolygon method too
7764660 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
temp prints
83d5c02 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
add 1000 comparison
b670c92 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
print val
15001c3 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
print val
8b1359e 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
print val
ce9957b 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
print
43c28ae 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
print
21ab0f1 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
more prints
04ce383 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
commit unused
a5b3289 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
hv
5c9bf34 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
hv
9018b83 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
hv
f42b02d 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
print temp
dcfb6ce 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
print temp before pre process
64e5949 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
print temp before pre process
b3cc4ea 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
solve several issues with data processing
1084de7 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
eliminate prints
cefd746 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
undo some (wrong?) changes
70f4429 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
@danielsanchezaran
delete blank space
70fed0e 
Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ktro2828 ktro2828 ktro2828 requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@danielsanchezaran @ktro2828