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

HYDRA-1103 : Update face selection shader to behave as an unlit shader #147

Merged

Conversation

roopavr-adsk
Copy link
Contributor

No description provided.

@roopavr-adsk roopavr-adsk self-assigned this Jul 11, 2024
VtValue(GfVec3f(faceSelectioncolor[0], faceSelectioncolor[1], faceSelectioncolor[2])*0.3f) });

// Emissive (component selection highlighting material should be independent of scene lighting)
node.parameters.insert(
Copy link
Collaborator

Choose a reason for hiding this comment

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

This will make the shader be visible when there is no lights, but I think the problem was also that it should not receive lighting at all, I don't think this resolves this problem, right ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The face selection should be visible even when there are no lights. Isn't that the required behavior? Selection highlighting should be independent of scene lighting. Maya VP2 face sel highlight is visible even when no scene lights are used(Lighting>Use No Lights)

Copy link
Collaborator

Choose a reason for hiding this comment

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

The 0.3 magic constant seems to appear in 3 places, lines 700, 705, and 707. Any reason for that choice? And why not factor it out?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The magic constant is based on VP2's shader behavior. Good idea to factor it out.

@roopavr-adsk
Copy link
Contributor Author

FYI : @lilike @tremblp @debloip @vlasovi @debloip-adsk

@roopavr-adsk roopavr-adsk added bug Something isn't working ready-for-merge Development process is finished, PR is ready for merge labels Jul 15, 2024
@roopavr-adsk roopavr-adsk merged commit 64339ae into dev Jul 15, 2024
10 checks passed
@roopavr-adsk roopavr-adsk deleted the roopavr/HYDRA-1103/faceSelectionIndependentOfSceneLighting branch July 15, 2024 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants