Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Edge direction visualization via shader #802

Merged
merged 4 commits into from
Oct 24, 2024
Merged

Conversation

tinxx
Copy link
Collaborator

@tinxx tinxx commented Oct 18, 2024

Implements a dedicated EdgeShader that handles color gradient. A vertex deformation effect can be configured and toggled that allows for a direction visualization. The effect depends on a correct UV mapping and uses the direction as present in our custom 3D edge meshes. The smoothness of the effect relies on several configurable parameters and the vertex density of the original mesh.

The shader is applied to our 3D edge meshes generated in SEESpline. The old direction visualization is replaced.

This MR fixes #789.

Implements a dedicated EdgeShader that handles color gradient.
A vertex deformation effect can be configured and toggled
that allows for a direction visualization. The effect depends
on a correct UV mapping and uses the direction as present
in our custom 3D edge meshes. The smoothness of the effect
relies on several configurable parameters and the vertex
density of the original mesh.
@tinxx tinxx added the Improvement Improvement of existing features label Oct 18, 2024
@tinxx tinxx requested a review from falko17 October 18, 2024 13:08
Copy link
Collaborator

@falko17 falko17 left a comment

Choose a reason for hiding this comment

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

Thank you for the PR, the effect looks very nice (and should be much more performant)! I only have a few minor suggestions below.

Assets/SEE/Game/Operator/EdgeOperator.cs Outdated Show resolved Hide resolved
Assets/SEE/Game/Operator/EdgeOperator.cs Outdated Show resolved Hide resolved
Assets/SEE/Game/Operator/EdgeOperator.cs Outdated Show resolved Hide resolved
Assets/SEE/GameObjects/SEESpline.cs Outdated Show resolved Hide resolved
Implement changes suggested by reviewer @falko17.
@tinxx
Copy link
Collaborator Author

tinxx commented Oct 21, 2024

Thank you for the review. I have applied the suggested changes.

Copy link
Collaborator

@falko17 falko17 left a comment

Choose a reason for hiding this comment

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

There's just two more comments regarding missing summary tags, but since these are trivial to fix, I'll just approve already. The CI will fail until #800 is merged, so we can ignore its output for this PR.

Assets/SEE/Game/Operator/EdgeOperator.cs Show resolved Hide resolved
Assets/SEE/GameObjects/SEESpline.cs Outdated Show resolved Hide resolved
@falko17 falko17 merged commit c4e3de9 into master Oct 24, 2024
9 of 11 checks passed
@tinxx tinxx deleted the edge-direction-visualization branch December 2, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Improvement of existing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Edge Direction Visualization via Shader
2 participants