-
Notifications
You must be signed in to change notification settings - Fork 155
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
feat(rhino): CNX-8348 CNX-8349 create rhino 8 connector converter projects #3101
feat(rhino): CNX-8348 CNX-8349 create rhino 8 connector converter projects #3101
Conversation
also updates preprocessor directives in converter
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.
Just a couple comments for now.
<!-- We are building for win-x64 on mac too, so these deps are not automatically copied/loaded --> | ||
<ItemGroup Condition="'$(Configuration)'=='Debug Mac' OR '$(Configuration)'=='Release Mac'"> | ||
<None Include="$(HOME)/.nuget/packages/avalonia.native/0.10.21/runtimes/osx/native/libAvaloniaNative.dylib" CopyToOutputDirectory="PreserveNewest" Visible="false" /> | ||
<None Include="$(HOME)/.nuget/packages/sqlitepclraw.lib.e_sqlite3/2.1.4/runtimes/osx-x64/native/libe_sqlite3.dylib" CopyToOutputDirectory="PreserveNewest" Visible="false" /> | ||
</ItemGroup> |
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.
Same as here. We may also want to look into dotnet publish
instead of dotnet build
as it has some differences in what is copied over to the final folder.
Objects/Converters/ConverterRhinoGh/ConverterRhinoGhShared/BrepEncoder.cs
Outdated
Show resolved
Hide resolved
…instead of OR conditions
Plus remove SDK warning from connectorRhino8
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.
Both Rhino and Grasshopper projects work on Windows (only).
Minimum version of Rhino 8 supported is 8.3
Description & motivation
Adds the Rhino8 connector (net7.0-windows) and converter projects. Successfully builds and loads plugin in Rhino 8.
Adds Grasshopper connector and converter projects for Rhino 8
Bumps Rhino dependencies to version 8.3
Changes:
To-do before merge:
Validation of changes: