Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Costmap #639
Costmap #639
Changes from 33 commits
8691b67
46a2479
ca4cb07
f79037b
01cca13
c61599b
c3bd12f
6eb8d2b
d945109
7c2c227
6d08489
db012d2
5bf78fa
0e6360d
95761c0
d82f70a
2ff1505
9a82b0c
782e22a
bf8c75f
de8ce87
234315f
6a448eb
c06b6f3
5e0f139
ea4471e
70aa5f0
d452ba4
8535c3e
da29226
4b4df3b
beae107
40acab3
41c4681
5342537
3039c9c
5c65ea4
36c1351
3e9cbb0
996575b
52a5bd8
4a4afa8
04654f3
e987b17
99bb271
227170c
6b74531
52df6b2
be439f3
e8b314c
caec28b
3f9c09c
57a5b43
973f25a
aea0665
6c8f040
3cfd481
890cb25
17b3a55
ef3c646
ce364ac
d555242
2be051e
572e818
a2592d4
bc8d1f3
a13a89d
60fa76b
59fb619
e5d9092
b877028
4cb242e
6b466e5
3e3b545
3e27986
1cbff99
1d277cd
cd678cf
3728c6d
fe8ab31
d794ae4
eaa6467
e5179c7
7e69d19
bb27616
40d8b3e
ca76f89
3effc25
fb9b018
f7c85ea
93fce06
87bbb85
054b377
a29f629
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure this iteration method bands vertically. What I mean is that it skips more horizontally rather than vertically. A good solution would be to make a custom iterator that is cognizant of the current row so it knows to skip them. Then we can use that iterator in std::for_each with par_unseq.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Im pretty sure you should leave the z component in the point here and only set it to zero once its in the map frame, since if the zed is rotated you end up removing parts of the true x and y components
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This makes sense. Should be fixed now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eigen::Vector3d normalInMap = normal_matrix.col(i).head<3>()
head
will grab the first three. No need to doEigen::Vector3d normal{n.x(), n.y(), n.z()};
later.This automatically de-homogenizes the vector. Note that we are not dividing by w which is usually done. But we know w=1 because we are multipliying by a SO3 matrix which is orthnormal and has det=1 so w won't change. Distance between points in your rigidbody is preservedi n other words
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same thing with naming here. Indicate the frame.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this normalization step necessary after just a single matrix multiplication? I assume the zed gives us unit vectors. although ig it probably doesn't slow things down much
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I just wasn't sure if the vectors in the pointcloud were unit vectors but it seems like they are.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can add an assert that the normal given by the zed is close to normalized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should remove all these starter project files right
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like it. Instead of doing this manually (idk what you did) you can do
git restore --source origin/integration starter_project
or whatever to revert whole folders to certain branches (in this case the target merge branch).