Skip to content

Jm/hotfix - #3

Merged
parkjinman98 merged 12 commits into
mainfrom
jm/hotfix
Aug 24, 2026
Merged

parkjinman98 merged 12 commits into
mainfrom
jm/hotfix

Conversation

@parkjinman98

Copy link
Copy Markdown
Contributor

No description provided.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 59abe08f9e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@parkjinman98

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7580bfc22f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@parkjinman98

Copy link
Copy Markdown
Contributor Author

@codex review

Please focus this review on the decoded YOLOv10 and pose postprocessing changes in 69331f0. Verify the documented ordered class-probability, selected-confidence, and box triplet for single-class face models; confirm multi-class outputs retain the existing class-wise dual top-k semantics; check invalid decoded values fail loudly; and verify compact pose extraction does not bypass non-end-to-end export behavior. Please ignore the already-resolved earlier threads unless the new implementation regresses them.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 69331f042d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@parkjinman98

Copy link
Copy Markdown
Contributor Author

@codex review

Please focus on the decoded-output validation changes in 5fe3e5a. Verify that all selected-confidence values are range-checked, that every above-threshold multi-class decoded box is validated before top-k truncation, and that compact decoded pose visibility logits are rejected when non-finite before sigmoid. Please also check that standard raw-output paths and legitimate decode-enabled YOLOv10 and pose outputs remain compatible.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5fe3e5aa0c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@parkjinman98

Copy link
Copy Markdown
Contributor Author

@codex review

Please focus on f3209e5. Verify that decoded YOLOv10 triplets preserve the standard fixed (B, 300, 6) tensor contract whenever e2e is disabled, while end-to-end behavior remains unchanged. Also verify that DFL-free decoded pose visibility values are checked before sigmoid and that raw output paths remain compatible.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Hooray!

Reviewed commit: f3209e58ae

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@parkjinman98
parkjinman98 merged commit 7195ddf into main Aug 24, 2026
3 checks passed
@parkjinman98
parkjinman98 deleted the jm/hotfix branch August 24, 2026 01:39
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.

1 participant