Skip to content

RTMO head deployment fix for non-square input sizes#3261

Open
jvasi wants to merge 3 commits intoopen-mmlab:mainfrom
jvasi:dev-1.x-rtmo-head-deployment-fix
Open

RTMO head deployment fix for non-square input sizes#3261
jvasi wants to merge 3 commits intoopen-mmlab:mainfrom
jvasi:dev-1.x-rtmo-head-deployment-fix

Conversation

@jvasi
Copy link
Copy Markdown

@jvasi jvasi commented Apr 3, 2026

Fixes the RTMO pose output during ONNX conversion for models with non-symmetrical input dimensions.

Motivation

RTMO deployment of models with custom input image size break during deployment.

Modification

The feature maps were incorrectly following an [N, C, W, H] format. This change ensures the dimensions are correctly transposed to [N, C, H, W].

This issue was previously masked when using symmetrical input resolutions (e.g., 640x640) where H and W are interchangeable, but caused output breakage for rectangular aspect ratios.

BC-breaking

No changing breaks to the backward compatibility as far as I am aware.

Use cases

No new use-cases

Checklist

Before PR:

  • I have read and followed the workflow indicated in the CONTRIBUTING.md to create this PR.
  • Pre-commit or linting tools indicated in CONTRIBUTING.md are used to fix the potential lint issues.
  • Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
  • New functionalities are covered by complete unit tests. If not, please add more unit tests to ensure correctness.
  • The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

  • CLA has been signed and all committers have signed the CLA in this PR.

vi3itor and others added 3 commits August 4, 2025 15:29
…-symmetrical input dimensions.

The feature maps were incorrectly following an [N, C, W, H] format. 

This change ensures the dimensions are correctly transposed to [N, C, H, W].
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 3, 2026

CLA assistant check
All committers have signed the CLA.

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.

3 participants