sanity_check_attributes bodies are mostly empty across visuals
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 25/100
- issue の種類
- バグ
- 明瞭さ
- 説明が足りない
- 活発さ
- 静か
- 技術スタック
- python
調査の方向性
Start by auditing the listed sanity_check_attributes and sanity_check_attributes_buffer methods across gsp/visuals, gsp/geometry, gsp/materials, and gsp/core, beginning with the pass implementations and shallow Buffer checks. Use the existing rgba8 enforcement in gsp/materials/mesh_basic_material.py and the related #20 Bug 6 context as reference. Done means each class has defined BufferType, count, and applicable shape or range validation that fails clearly at construction or the buffer boundary.
索引モデルが issue の本文から書いたものです。
説明
Short tracking issue — most sanity_check_attributes / sanity_check_attributes_buffer static methods across the visuals/geometry/material/texture classes are either pass or limited to isinstance(x, Buffer) checks. As a result, malformed input (wrong dtype, wrong shape, mismatched counts) crashes deep in numpy at render time instead of failing clearly at construction or at the buffer boundary.
Audit (as of mesh_ng):
gsp/visuals/mesh.py—sanity_check_attributesispass;_bufferform only delegates.gsp/visuals/markers.py— bothpass.gsp/visuals/paths.py— bothpass.gsp/geometry/mesh_geometry.py— bothpass.gsp/visuals/pixels.py,points.py,segments.py,texts.py—_bufferonly has shallowisinstance(_, Buffer)checks; no shape/dtype/count validation.gsp/visuals/image.py,gsp/core/texture.py— partial type checks only.gsp/materials/mesh_basic_material.py—_buffernow enforces rgba8 on face/edge colors (#20 Bug 5); the non-buffer form is stillpass.
What good would look like (per class):
- Buffer
BufferTypematches the expected encoding (e.g. positions →vec3, indices →uint32, colors →rgba8). - Buffer count consistency between related attributes (e.g. material color count ∈ {1, vertex_count, face_count}).
- Optionally O(N) range checks (e.g.
max(indices) < vertex_count) at the buffer boundary.
Related: this is the larger Bug 6 from #20, broadened beyond Mesh.
This is just a placeholder to come back to — no fix scoped yet.
- 主要言語
- Python
- スター
- 2
- フォーク
- 0
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
vispy/GSP_API のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 65/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
stephrobert/dsoxlab#238 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
sublimehq/package_control#1780 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
nwg-piotr/nwg-displays#145 ·