Skip to content

Vertexまわりの実装改善#232

Draft
Siro256 wants to merge 10 commits intomasterfrom
feat/vertex-improvement
Draft

Vertexまわりの実装改善#232
Siro256 wants to merge 10 commits intomasterfrom
feat/vertex-improvement

Conversation

@Siro256
Copy link
Copy Markdown
Member

@Siro256 Siro256 commented Mar 22, 2026

No description provided.

@Siro256
Copy link
Copy Markdown
Member Author

Siro256 commented Mar 22, 2026

アスペクト比の計算とかそういう細かいのがrenderer/worker/に散らばってるので、適切な場所に配置したい

@Siro256 Siro256 force-pushed the feat/vertex-improvement branch from d0f2593 to 1ece455 Compare March 23, 2026 18:22
@Siro256
Copy link
Copy Markdown
Member Author

Siro256 commented Mar 23, 2026

[f32; 2]とかを使いまくってるので、Vertexを使うように置き換えていきたい
(実は様々を統合するとVertexという名前が不適切になる?)

@Siro256
Copy link
Copy Markdown
Member Author

Siro256 commented Mar 27, 2026

@yanorei32 まかせた

@Siro256
Copy link
Copy Markdown
Member Author

Siro256 commented Mar 28, 2026

  • BoundingBox、Rectangleのほうが良さそう?
  • BoundingBox::gl_top_rightとか、CoordTypeが変わると嘘になりうるのでfirst_quadrantとかにしたほうが良さそう?
  • 同様にBoundingBox::value_top_leftもCoordTypeによって嘘になりうる

@Siro256
Copy link
Copy Markdown
Member Author

Siro256 commented Mar 29, 2026

メモ: encapsulate merge

@Siro256 Siro256 closed this Mar 29, 2026
@Siro256 Siro256 reopened this Mar 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant