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

surface orientation rules with ImportInstance.ByGeometries #2303

Open
ksobon opened this issue Nov 17, 2014 · 4 comments
Open

surface orientation rules with ImportInstance.ByGeometries #2303

ksobon opened this issue Nov 17, 2014 · 4 comments

Comments

@ksobon
Copy link
Contributor

ksobon commented Nov 17, 2014

I am not sure I understand how this node is working because it tends to produce some weird results. I am using ImportInstance.ByGeometries to import geometry from Grasshopper to Revit. It seems to me that some faces are mirrored horizontally and some are rotated. Anyone else having that same issue?

capture
capture1

is this @pboyer @HLP or @kronz question?

ps. I tested this with Revit UI imports and same results. I tried going through In-place mass and then import. No luck.

@ptierney
Copy link
Contributor

Are you importing with the SAT importer? Can you share any of your geometry or Dynamo files?

@ikeough
Copy link

ikeough commented Nov 18, 2014

@pboyer The UV space seems to be mirrored? We had seen certain types of Revit surfaces that had their normals reversed.

@ksobon Does the same happen for curved surfaces?

@ksobon
Copy link
Contributor Author

ksobon commented Nov 18, 2014

@ikeough @pboyer I am importing it from GH using Mantis Shrimp. Its a simple brep. I also tried baking and importing via Revit UI as an SAT file. I tried a curved surface and had no trouble with it. I will test further, and post files when I can. If you want to re-create it Mantis Shrimp is on Package Manager just stream a box from GH to Dynamo(RH Brep to DY Polysurface) then extract all faces and use FamilyInstance.ByGeometries to import them in as 6 separate instances (i need that to keep UV re-mapping sane).

@Racel
Copy link
Contributor

Racel commented Oct 30, 2017

@ksobon - Is this still an issue? Not sure what version you were working with. Can you let us know.

@johnpierson johnpierson transferred this issue from DynamoDS/Dynamo Dec 14, 2018
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

No branches or pull requests

4 participants