Autodesk / Autodesk/revit-ifc

Railing Geometry adopted From Active View

Open
#425 1 comment 0 reactions 0 assignees View on GitHub
API enhancement export geometry problem Triage
Dominant language
C#
Stars
637
Forks
225
PR merge metrics
No merged PRs in 30d

Description

Hi, I was requested today to help trouble shoot a revit IFC export that is stalling.
It has lots of railings, which have spheres as a connection between baluster and railing.
![220303 railing spheres](https://user-images.githubusercontent.com/791757/156559622-67cbfcde-70a7-4a1f-bce6-a694903874ab.png)

I suggested to disable their display in a course detail, and export geometry based on active view.
But the setting doesn't work for railings. It calls the method here:
https://github.com/Autodesk/revit-ifc/blob/65edb501e422d415ceb6fd28e99b76b50990662f/Source/Revit.IFC.Export/Utility/GeometryUtil.cs#L385

This doesn't account for ExporterCacheManager.ExportOptionsCache.ActiveView
I'd suggest it should?
I made a quick test by refactoring to use the active view, this made a dramatic difference to the export time which did succeed with a much small file size.

Thanks,

Jon

Contributor guide

Open the contributing guide

Research direction

Start in Source/Revit.IFC.Export/Utility/GeometryUtil.cs around line 385 and inspect how ExporterCacheManager.ExportOptionsCache.ActiveView is used for geometry export. Reproduce the railing export with active-view geometry enabled, then verify that railing connection spheres follow the setting and the export completes with reduced geometry.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
computer-graphics
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.