False positive FES warning for loadModel(path, callback)
メンテナーはふだん 3 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 62/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- javascript
調査の方向性
この警告は loadModel() に対する Friendly Error System の引数チェックから出ています。まずそこから始めて、2番目の引数に文字列を期待する分岐を見つけてください。loadModel(path, true) を扱っていた PR #9286 の修正と比較し、コールバック形式も同じチェックを通るようにしてください。有効なコールバックを渡した loadModel(path, callback) が FES の警告を出さず、loadModel(path, true) が従来どおり動作すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Most appropriate sub-area of p5.js?
- Accessibility
- Color
- Core/Environment/Rendering
- Data
- DOM
- Events
- Image
- IO
- Math
- Typography
- Utilities
- WebGL
- WebGPU
- p5.strands
- Build process
- Unit testing
- Internationalization
- Friendly errors
- Other (specify if possible)
p5.js version
2.3.4
Web browser and version
Google Chrome 155.0.8059.40 (Official Build) (64-bit)
Operating system
Windows 11 Version 25H2 (Build 26200.9457)
Steps to reproduce this
Steps:
- Open the p5.js Web Editor.
- Upload a valid
teapot.objfile into anassetsfolder in the sketch. - Run the following sketch and check the console.
Snippet:
let teapot;
function setup() {
createCanvas(400, 400, WEBGL);
loadModel("/assets/teapot.obj", function (modelData) {
teapot = modelData;
console.log("Model loaded successfully:", modelData);
});
}
function draw() {
background(220);
orbitControl();
normalMaterial();
if (teapot) {
scale(2);
model(teapot);
}
}
Expected behavior
Calling loadModel(path, callback) with a valid callback should not produce a Friendly Error System (FES) warning.
Actual behavior
The model loads and renders successfully, and the callback executes. However, FES reports:
p5.js says: Expected string at the second parameter in loadModel().
This appears to be a false positive because the callback is a valid second argument for this overload.
Additional context
This is a follow-up to Issue #9176 and it's fix PR #9286 , which addressed the false-positive FES warning for loadModel(path, true).
The callback overload loadModel(path, callback) still produces a similar warning after that fix. I reproduced this behavior in the p5.js Web Editor using a local OBJ file.
I'd like to work on this issue if approved. Could a maintainer please review it and assign it to me if appropriate?
Screenshot of Issue Visually
- 主要言語
- JavaScript
- スター
- 24.1k
- フォーク
- 3.9k
- 平均マージ
- 2日 20時間
- マージ済み PR(30日)
- 37
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
processing/p5.js のほかの issue
-
[p5.js 2.0+ Bug Report]: touches[0].x lags one pointer event behind while dragging対応中かも @titanstride が 2 日前に担当しました。 オープンArea:Events p5.js 2.0+
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
processing/p5.js#9305 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
loadModel() options break without fileType and ignore flipU/flipV対応中かも @saniyaa-17 が 2 日前に担当しました。 オープンArea:WebGL p5.js 2.0+
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
processing/p5.js#9302 · コメント 2 件 ·
メンテナーはふだん 3 日以内に返信
-
[p5.js 2.0+ Bug Report]: key docs needs updating対応中かも @MannXo が 2 日前に担当しました。 オープンHelp Wanted
難易度 1/5 1時間未満 初心者へのやさしさ 62/100
processing/p5.js#9296 · コメント 5 件 · リアクション 1 件 ·
メンテナーはふだん 3 日以内に返信
-
Add unit tests for noiseDetail()対応中かも @harshiltewari2004 が 5 日前に担当しました。 オープンArea:Math Enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
processing/p5.js#9253 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 3 日以内に返信
-
[p5.js 2.0+ Bug Report]: storeItem() validation branches are missing `return`, so bad input throws a TypeError or is stored anyway対応中かも @iUjjwalRaj が 13 日前に担当しました。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
processing/p5.js#9242 ·
メンテナーはふだん 3 日以内に返信
processing/p5.js の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
maintenance
難易度 1/5 1時間未満 初心者へのやさしさ 75/100
alphagov/accessible-autocomplete#826 ·
メンテナーはふだん 4 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
olahol/react-tagsinput#229 ·
-
[BUG] npm pkg delete removes the first array item for a nonexistent property対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
jonathanong/no-mistakes#2260 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信