Skip to content

Conversation

@cieplypolar
Copy link
Collaborator

@cieplypolar cieplypolar commented Nov 21, 2025

  • wgsl resolution test for lines combinations example (tricky, because there are a lot of options)

Closes #1930.

@github-actions
Copy link

github-actions bot commented Nov 21, 2025

pkg.pr.new

packages
Ready to be installed by your favorite package manager ⬇️

https://pkg.pr.new/software-mansion/TypeGPU/typegpu@582630b4069c4b7d3b05d6148aac3876728dfed0
https://pkg.pr.new/software-mansion/TypeGPU/@typegpu/noise@582630b4069c4b7d3b05d6148aac3876728dfed0
https://pkg.pr.new/software-mansion/TypeGPU/unplugin-typegpu@582630b4069c4b7d3b05d6148aac3876728dfed0

benchmark
view benchmark

commit
view commit

@cieplypolar cieplypolar marked this pull request as ready for review November 25, 2025 16:03
Copy link
Contributor

@aleksanderkatan aleksanderkatan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Do we plan on keeping these examples as dev only?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does not look intended, did it look like this before the example stopped working?

image

Also, the last two test cases disappear when the segment becomes too short.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I know about these issues. I have this new PR open which rewrites the implementation, then I'll think about what to do in these specific cases. It is not completely obvious to me what strategy is visually best yet.

Copy link
Contributor

@deluksic deluksic Nov 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh wait, this exact issue I have not seen before. But maybe look at the new implementation PR and see if you can rebase your changes. I have already aligned (all?) the implementations with recent main.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had another look, and indeed this happens when you try using "arrow" caps. They are kinda fiddly in terms of cases where they work nicely. When there is such a short and thick segment, the arrow is way bigger than it, causing the issue you are seeing. So in this case, yes it is expected.

@iwoplaza
Copy link
Collaborator

Do we plan on keeping these examples as dev only?

Yes, until we publish the @typegpu/geometry package.

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.

fix: Fix 'Lines Combinations' and 'Wind Map' examples

5 participants