Camera parameter table (arParamDisp) printed regardless of the log level
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Read the loadCamera() call to arParamDisp() in WebARKit/WebARKitTrackers/WebARKitNFT/ARToolKitNFTCore.cpp and the logging behavior described for include/ARUtil/log.h and lib/SRC/AR/paramDisp.c. The issue leaves open whether to gate the output, keep it unconditional, or expose it separately, so clarify that decision before starting implementation. If gating is chosen, add stdout tests for INFO and WARN and review the CAMERA_DUMP filters in the two runtime-settings tests; done means the chosen behavior and tests agree.
索引モデルが issue の本文から書いたものです。
説明
Problem
ARToolKitNFTCore::loadCamera() (WebARKit/WebARKitTrackers/WebARKitNFT/ARToolKitNFTCore.cpp, around line 445) calls arParamDisp(). That prints the camera parameter table through ARLOG, which is a plain printf with no log-level check (include/ARUtil/log.h, lib/SRC/AR/paramDisp.c):
--------------------------------------
SIZE = 640, 480
Distortion factor: k1=..., k2=..., p1=..., p2=...
fx=... fy=... x0=... y0=... s=...
<camera matrix rows>
--------------------------------------
So the table is printed whatever arLogLevel / webarkitLogLevel is set to. Since 0.12.0 every other start-up line follows the log level. jsartoolkitNFT can now set it before the core is constructed, for webarkit/jsartoolkitNFT#677. With logLevel = Error, this table is the only native start-up output left.
To decide
It is still open whether the table should be silenced. It is useful when debugging a camera calibration, and some users may rely on seeing it.
- Gate it on the level. Call
arParamDisp()only whenarLogLevel <= AR_LOG_LEVEL_INFO. It keeps printing by default (INFO) and disappears at WARN or above. - Keep it unconditional, and document that it is printed whatever the level. jsartoolkitNFT's tests already tolerate it explicitly.
- Expose it separately, for example as a flag or a debug-only call.
If it is gated
- Change only the call site in the core.
paramDisp.cis vendored ARToolKit code. - Add a test that captures stdout at INFO and at WARN.
- In jsartoolkitNFT, delete the
CAMERA_DUMPfilter from the start-up silence test intests/vitest/runtime-settings.test.tsandtests/node/runtime-settings.test.js. A comment marks it as covering this gap.
To be bundled with #107, #110 and #111 in a single 0.12.1 release.
- 主要言語
- C++
- スター
- 21
- フォーク
- 8
- 平均マージ
- 8時間 1分
- マージ済み PR（30日）
- 21
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
webarkit/WebARKitLib のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
webarkit/WebARKitLib#114 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 93/100
webarkit/WebARKitLib#107 ·
メンテナーはふだん 1 日以内に返信
-
configure(): an explicit logLevel equal to arLogLevel does not sync a divergent webarkitLogLevelオープン
難易度 5/5 1週間以上 初心者へのやさしさ 38/100
webarkit/WebARKitLib#111 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
webarkit/WebARKitLib#110 ·
メンテナーはふだん 1 日以内に返信
-
Sync workflow: handle a conflicting sync branch with no open PR, and PRs the workflow did not openオープンCI enhancement
難易度 3/5 半日 初心者へのやさしさ 45/100
webarkit/WebARKitLib#92 ·
メンテナーはふだん 1 日以内に返信
webarkit/WebARKitLib の issue をすべて見る
似ている issue
-
area:qmc bug
難易度 2/5 1〜3時間 初心者へのやさしさ 69/100
メンテナーはふだん 3 日以内に返信
-
bug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
ASGC-Game-Jam/asgc-gamejam-2026#563 ·
メンテナーはふだん 1 日以内に返信
-
area/ysql kind/bug priority/medium status/awaiting-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
yugabyte/yugabyte-db#37523 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
Cockatrice/Cockatrice#7896 ·
メンテナーはふだん 1 日以内に返信
-
mlir
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
llvm/llvm-project#241070 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信