Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

[Accuracy] MoGe selected convolutions replace Native replicate padding with zeros

クローズ
#1,727 コメント 0 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@chaofengw-nv がすでに取り組んでいます。

2026年10月15日 から。

  • #1734 @chaofengw-nv による — マージされずにクローズ
  • #1822 @chaofengw-nv による — オープン

評価

この issue はまだ評価されていません。

説明

bug

Affected area

Inference output or task execution

TensorRT-Model-Connect version

Diagnostic snapshot 174df6571e2152ad95edeb1a5c1c30b1257ecb6f; affected family production files also present at cef761444d2525ad00a86cfe0413157801d175ab.

Installation method

Development container / source build.

Model and revision

Ruicheng/moge-2-vitl @ 39c4d5e957afe587e04eec59dc2bcc3be5ecd968

Environment

  • NVIDIA GB300, CUDA 13.0, TensorRT 11.2.0.113 for the prior model controls.
  • CPU regression uses the installed TensorRT Python API without CUDA inference when applicable.

Minimal reproducer

A hardcoded set of 24 neck/points/mask convolutions skips the official replicate-padding operation and substitutes convolution zero padding. These are different boundary conditions and change valid border predictions.

With an AIPerf environment configured as described in apps/aiperf_qual/README.md, run the family profile before and after the production change:

trtmc-aiperf-qual run --profile moge-2-vitl --environment "$E" --out out/moge-2-vitl

The exact prior replay outcomes are listed below. This submission does not claim that this public CLI command was rerun on the final PR head.

For local CPU contract coverage (not a model-parity reproducer):

python -m pytest -q families/moge/tests/test_selective_zero_padding.py

Observed behavior

HUMAN REVIEW REQUIRED: the three existing tests explicitly demand the conflicting zero-padding behavior and still fail after this correction. No tests, gates or thresholds were changed.

Earlier GB300 full 50-image geometry control: restoring replicate padding improved 47/50 -> 49/50 with original mask IoU/depth criteria. One depth-error case remains above the original 1% limit.

Expected behavior

Remove the selected zero-padding override and retain each convolution caller's replicate-padding contract.

Relevant logs

Specific affected source: families/moge/model.py. No private logs or model assets are attached.

Impact and scope

Restore Native padding only; do not relax the 1% depth criterion or claim complete MoGe parity.

Draft only: existing test expectations require a maintainer decision; no passing-CI claim. Remaining depth error and performance are not fixed.

Submission checks

  • Searched open and closed issues for this root cause; no duplicate found.
  • This is not a security vulnerability.
  • Removed credentials, private URLs, internal paths and restricted artifacts.
主要言語
Python
スター
276
フォーク
73
平均マージ
1日 7時間
マージ済み PR(30日)
166

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

NVIDIA/TensorRT-Model-Connect のほかの issue

NVIDIA/TensorRT-Model-Connect の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。