-
Notifications
You must be signed in to change notification settings - Fork 636
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
Helix upgrade to Dynamo 2.7 #10510
Merged
Merged
Helix upgrade to Dynamo 2.7 #10510
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
c7e9143
Move to Helix 2.5.1 and SharpDX 4.2.0
mike-d-adsk fba0ef7
Merge remote-tracking branch 'origin' into helix-upgrade
mike-d-adsk 73b36dc
Re-implement zoom in/out buttons
mike-d-adsk ae98cf6
Upgrade Helix to 2.9.0
mike-d-adsk d049ecc
Re-implement headlight outside of shader
mike-d-adsk 4b04801
Remove Helix shader pipeline customizations, use default shaders (WIP)
mike-d-adsk ae0c2b0
Merge remote-tracking branch 'origin' into helix-upgrade
mike-d-adsk 187e375
merge with master branch
aparajit-pratap 3fd7d03
restore updating colors for selected and frozen node states
aparajit-pratap f5d42e6
Merge branch 'master' of github.com:DynamoDS/Dynamo into helix-upgrade
aparajit-pratap 696c188
code cleanup
aparajit-pratap c953502
Use Viewport3DX.Items, enable geometry -> node selection
aparajit-pratap ba14838
Merge branch 'master' of github.com:DynamoDS/Dynamo into helix-upgrade
aparajit-pratap 9f639bf
Merge branch 'upMaster' into helix-upgrade
mjkkirschner d1f0344
Helix upgrade basics (#10326)
mjkkirschner fa0f1bc
Merge branch 'helix-upgrade' of https://github.com/DynamoDS/Dynamo in…
mjkkirschner 58d5116
Helix upgrade PR to merge the changes from the master branch (#10388)
reddyashish e81f34a
[DYN-2341] Labels on helix-update branch cause slow down (#10399)
reddyashish 7749e31
DYN-2406: better handle pts/lines when selecting, freezing, and isola…
mjkkirschner 9d5917d
fix alpha value for mesh colors in isolation mode. (#10405)
mjkkirschner 6d89d72
Helix upgrade mastermerge (#10465)
mjkkirschner 65612e0
Mesh display using shaders (#10462)
mjkkirschner fe42f38
Headlight bound to Camera LookDirection (#10466)
mjkkirschner b9d608f
special render packages should render with materials colors only (#10…
mjkkirschner 96126de
Update ProtoGeometry.config (#10483)
mjkkirschner 98b3fd9
Avoid rendering Gizmo multiple times on rotate (#10479)
mmisol d77d640
add 3d graphics image comparison tests (#10487)
mjkkirschner 9d4bdff
target shader model 4 to avoid crash when user does not have GPU. (#1…
mjkkirschner 1c4c98f
fix shader interpolation and make image comparison tests more toleran…
mjkkirschner 08deea9
Merge branch 'master' into helix-upgrade
QilongTang a00047f
Merge branch 'master' into helix-upgrade
QilongTang 7faf055
Handle memory outage on render (cherry pick) (#10569)
mmisol 9af2463
Add Image Comparison Test for Surface. (#10579)
zeusongit a9f5bef
Merge branch 'master' into helix-upgrade
QilongTang 4d713f2
Remove Dup entry
QilongTang fbab4a9
Comments
QilongTang 9df9ff4
Comments
QilongTang a1465af
Comments
QilongTang f594a68
Revert test file name change
QilongTang File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
<?xml version="1.0"?> | ||
<ProtoGeometryConfiguration xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"> | ||
<GeometryFactoryFileName>libg_221_0_0/LibG.ProtoInterface.dll</GeometryFactoryFileName> | ||
<GeometryFactoryFileName>libg_225_0_0/LibG.ProtoInterface.dll</GeometryFactoryFileName> | ||
</ProtoGeometryConfiguration> |
This file contains 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
This file contains 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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if we should instead aim for the latest version of Helix, which is 2.11. I'm pretty sure I heard @mjkkirschner mention that it worked just as well as 2.9 with Dynamo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@QilongTang I didn't notice this was .net standard, does this have the same issue as the pythonNet net standard stuff and bring in a ton of type forwarding system binaries? We may want to try updating dynamo to 4.7.2 ASAP.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont see a tons of added system binaries in the built folder, but im not sure about the latest .11 version