[p5.js 2.0+ Bug Report]: TRIANGLES Requires Multiples of 3
メンテナーはふだん 2 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 66/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- javascript
- 領域
- frontend
調査の方向性
まず、リンクされている p5.js エディタのスケッチで報告されたケースを再現します。beginShape(TRIANGLES) を呼び出し、4つの頂点を追加してから endShape() を呼び出します。追加の頂点が無視されていた p5.js 1 での結果と比較します。p5.js 2 が追加の頂点を実行時エラーなしで処理し、完全な三角形を描画できれば完了です。
索引モデルが 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
No response
Operating system
No response
Steps to reproduce this
Steps:
Creating a custom shape using the beginShape(TRIANGLES) option in v2.0+ requires the number of vertices to be exactly a multiple of 3 or it will hit a runtime error and not draw anything. The error is not a particularly friendly one: TypeError: can't access property "position", v1 is undefined
In p5 V1, extra vertices that could not be wrangled into a triangle (e.g. a shape with 4 vertices) were simply ignored.
I think the V1 method makes a lot more sense, and we should go back to ignoring extra vertices. This would also maintain some continuity for people porting V1 code to V2.
Snippet:
https://editor.p5js.org/lewi0622/sketches/lnZLG0GhN
beginShape(TRIANGLES);
vertex(random(width), random(height));
vertex(random(width), random(height));
vertex(random(width), random(height));
//uncommment fourth vertex to see issue in v2+, not in an issue in v1
// vertex(random(width), random(height));
endShape();
- 主要言語
- JavaScript
- スター
- 24.2k
- フォーク
- 3.9k
- 平均マージ
- 2日 17時間
- マージ済み PR(30日)
- 36
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
processing/p5.js のほかの issue
-
Area:WebGL p5.js 2.0+ p5.strands
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
processing/p5.js#9317 · コメント 4 件 ·
メンテナーはふだん 2 日以内に返信
-
Area:DOM Good First Issue p5.js 2.0+
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
processing/p5.js#9316 · コメント 4 件 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
-
[p5.js 2.0+ Bug Report]: touches[0].x lags one pointer event behind while dragging再び着手できるかも @titanstride が 9 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンArea:Events p5.js 2.0+
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
processing/p5.js#9305 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
loadModel() options break without fileType and ignore flipU/flipV再び着手できるかも @saniyaa-17 が 9 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンArea:WebGL p5.js 2.0+
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
processing/p5.js#9302 · コメント 2 件 ·
メンテナーはふだん 2 日以内に返信
-
[p5.js 2.0+ Bug Report]: key docs needs updating再び着手できるかも @MannXo が 9 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンHelp Wanted
難易度 1/5 1時間未満 初心者へのやさしさ 62/100
processing/p5.js#9296 · コメント 6 件 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
processing/p5.js の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
type/bug
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
flarum/framework#5301 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
AOSSIE-Org/BabyNest#284 ·
メンテナーはふだん 1 日以内に返信
-
status:untriaged
難易度 1/5 1時間未満 初心者へのやさしさ 86/100
midnightntwrk/servicedesk#275 ·
メンテナーはふだん 1 日以内に返信