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

Edge case error on convert from Revit roof to structural panel #1287

Open
pawelbaran opened this issue Dec 12, 2022 · 1 comment
Open

Edge case error on convert from Revit roof to structural panel #1287

pawelbaran opened this issue Dec 12, 2022 · 1 comment
Assignees
Labels
type:bug Error or unexpected behaviour

Comments

@pawelbaran
Copy link
Member

pawelbaran commented Dec 12, 2022

Description:

Following this comment by @vietle-bh, an error occurs on pull of the roof below from the standard test file.

image

The bottom of the issue lies in the fact that BH.Engine.Structure.Create.Panel method does not take geometrical tolerance as input, causing the Revit outline to be considered as not closed (even though it is acc. to Revit). Issue on BHoM_Engine raised under BHoM/BHoM_Engine#2961 - once it gets resolved, Revit tolerance will need to be passed in the line below (and in all other panel creation calls).

result.Add(BHS.Create.Panel(planarSurface.ExternalBoundary.ITranslate(translation), internalBoundaries, property2D, null, hostObject.Name));

@pawelbaran
Copy link
Member Author

I have debugged this issue after BHoM/BHoM_Engine#2961 got fixed and it still occurs - after digging deeper, it turned out that the problem lies in wrong face geometry being queried from the roof. There are two aspects here:

  • some faces of the element are nonplanar
  • some planar faces have hermite splines and nurbs curves as edges, which fail to convert

First point can be addressed by adding a meaningful message (to be PRed after the feature freeze), the other is a deeper issue that requires more work, already captured under #798.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Error or unexpected behaviour
Projects
None yet
Development

No branches or pull requests

2 participants