Skip to content

Add schema for head & hand pose#17

Merged
jiwenc-nv merged 4 commits intomainfrom
jiwenc/head_hand_pose
Dec 18, 2025
Merged

Add schema for head & hand pose#17
jiwenc-nv merged 4 commits intomainfrom
jiwenc/head_hand_pose

Conversation

@jiwenc-nv
Copy link
Collaborator

No description provided.

@jiwenc-nv jiwenc-nv marked this pull request as ready for review December 18, 2025 00:12
1/ It should be a struct (fixed layout) than a table, so that it can be easily composed into array in other tables
2/ It should use point and quaternion rather than tensors
@jiwenc-nv jiwenc-nv force-pushed the jiwenc/head_hand_pose branch from 7c067ac to a6697be Compare December 18, 2025 01:20
@@ -0,0 +1,53 @@
// SPDX-FileCopyrightText: Copyright (c) 2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add new headers to cmakelists for easier access/search from IDE

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you mean list the header file explicitly in the CMakeLists.txt?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, along with cpp

@jiwenc-nv jiwenc-nv force-pushed the jiwenc/head_hand_pose branch from a6697be to 09bcc8c Compare December 18, 2025 06:02
@jiwenc-nv jiwenc-nv changed the title Add head hand pose Add schema for head & hand pose Dec 18, 2025
@jiwenc-nv jiwenc-nv force-pushed the jiwenc/head_hand_pose branch from 09bcc8c to 65c68d2 Compare December 18, 2025 06:25
@jiwenc-nv jiwenc-nv force-pushed the jiwenc/head_hand_pose branch from 65c68d2 to d8a2061 Compare December 18, 2025 06:34
@jiwenc-nv jiwenc-nv merged commit 23ad50f into main Dec 18, 2025
4 checks passed
@jiwenc-nv jiwenc-nv deleted the jiwenc/head_hand_pose branch December 18, 2025 18:36
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