Skip to content

Conversation

@NullClone
Copy link
Contributor

HolisticLandmarkerでoutputFaceBlendshapesをtrueに設定した際、トラッキング対象の顔が検出されなくなるとクラッシュする問題を修正しました。

問題点

Getを呼び出す前にこのパケットが空であるかチェックを行っていなかったため、クラッシュが発生していました。

F0000 00:00:1762518056.291455   30580 packet.h:818] Packet::Get() failed: Expected a Packet of type: class mediapipe::ClassificationList, but received an empty Packet.

修正内容

faceBlendshapesPacketがnullまたはIsEmptyであるかを確認する処理を追加しました。

faceBlendshapes.categories?.Clear()が適切かどうか自分は詳しく分からないため、ご意見をいただけると助かります!

@homuler homuler self-requested a review November 8, 2025 01:42
@homuler homuler added the ci:run Run CI label Nov 8, 2025
@github-actions github-actions bot removed the ci:run Run CI label Nov 8, 2025
Copy link
Owner

@homuler homuler left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM 🚀

@homuler homuler merged commit 41886bd into homuler:master Nov 8, 2025
13 checks passed
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