You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
In 2.7 there was a regression with isolateSelection, but I also found there was a long standing issue with overlapping geometry in some cases not highlighting correctly even during normal selection.
this pr does the following:
inside the dynamo mesh vertex shader - selected geometry is moved along the normal of each vertex by .01 - this is large enough to prevent z-fighting even when zoomed outed until the geometry is no longer visible.
geometry which requires vertex colors maintains its original movement of .0001
if a vertex is both vertex colored and selected it moves by .0101
visualization tests are added for two spheres in exactly the same location in isolation mode and regular selection mode.
The reason that we need an additional increment for selected geometry is because I found most mesh geometry we generate has white vertex colors so all verts are moved slightly... essentially nothing special was being done for selected geometry. These colors were applied in LibG many years ago, and I don't think it is worth the risk to modify that at this point.
For those who are curious I found that the reason this worked in 2.6 for isolated geometry, which was that after every interaction we would resort all geometry in the scene so that the selected geometry was rendered last... it's great to be able to skip this given that this sorting could be slow if there was a lot of geometry.
…n highlight. (#11037)
* this works, but ideally it would be done in the shader to avoid this sorting
* update shader to make selected geometry more prominent than vertColored geo
new tests
* move camera closer
regen images
disable update image define
remove unneccesary sorting
* fix copy paste error
…n highlight. (#11037) (#11041)
* this works, but ideally it would be done in the shader to avoid this sorting
* update shader to make selected geometry more prominent than vertColored geo
new tests
* move camera closer
regen images
disable update image define
remove unneccesary sorting
* fix copy paste error
Co-authored-by: Michael Kirschner <mjk.kirschner@gmail.com>
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
In 2.7 there was a regression with isolateSelection, but I also found there was a long standing issue with overlapping geometry in some cases not highlighting correctly even during normal selection.
this pr does the following:
inside the dynamo mesh vertex shader - selected geometry is moved along the normal of each vertex by
.01
- this is large enough to prevent z-fighting even when zoomed outed until the geometry is no longer visible..0001
.0101
visualization tests are added for two spheres in exactly the same location in isolation mode and regular selection mode.
The reason that we need an additional increment for selected geometry is because I found most mesh geometry we generate has white vertex colors so all verts are moved slightly... essentially nothing special was being done for selected geometry. These colors were applied in LibG many years ago, and I don't think it is worth the risk to modify that at this point.
For those who are curious I found that the reason this worked in 2.6 for isolated geometry, which was that after every interaction we would resort all geometry in the scene so that the selected geometry was rendered last... it's great to be able to skip this given that this sorting could be slow if there was a lot of geometry.
Declarations
Check these if you believe they are true
*.resx
files