Skip to content

Update convex hull package from NuGet used in Tessellation - #7811

Merged
aparajit-pratap merged 10 commits into
DynamoDS:masterfrom
aparajit-pratap:updateConvexHull
Apr 24, 2017
Merged

aparajit-pratap merged 10 commits into
DynamoDS:masterfrom
aparajit-pratap:updateConvexHull

Conversation

@aparajit-pratap

Copy link
Copy Markdown
Contributor

Purpose

Noticed that the MIConvexHull package we were using for Tessellation nodes was outdated so I've updated the package from NuGet.

Declarations

Check these if you believe they are true

  • The code base is in a better state after this PR
  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
    There are no existing tests for Voronoi and Dealaunay nodes so these should be added here as well
  • User facing strings, if any, are extracted into *.resx files
  • All tests pass using the self-service CI.
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning, and are documented in the API Changes document.

…ynamoDS#7623)

* update LibG binaries

* cherry-pick: update preloader for ASM223

* remove support for LibG220

* update test frameworks to preload appropriate ASM version

* remove ASM220 library version

* update LibG binaries for newer ASM 223 version used in Revit2018

* fix flaky test case

* recover from geometry exceptions thrown during manipulator creation

* fix for graph re-execution after changing scaling
@@ -0,0 +1,1358 @@
<?xml version="1.0"?>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not sure if we need to check-in this file.

@sharadkjaiswal

Copy link
Copy Markdown
Contributor

Thanks @aparajit-pratap this looks good.

@aparajit-pratap

Copy link
Copy Markdown
Contributor Author

Thanks @sharadkjaiswal

@aparajit-pratap
aparajit-pratap merged commit 2dc648c into DynamoDS:master Apr 24, 2017
@aparajit-pratap
aparajit-pratap deleted the updateConvexHull branch April 24, 2017 04:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants