BIM-Tools / SketchUp-IFC-Manager

IFC data manager and exporter for SketchUp.
GNU General Public License v3.0
59 stars 20 forks source link

v5.0.2 hierarchy issue #51

Closed valleyarchitect closed 10 months ago

valleyarchitect commented 1 year ago

Hi,

I've been trying v5.0.2 in SU23.1 & am impressed by the texture/color transfer improvements. However, I noticed an issue in hierarchy. Here is the SU file I was working on & the resulting ifc file, plus the same model exported using v3.4.7 of your plugin where there are no issues concerning the hierarchy. I've also added screenshots so you know what options were active in the preferences before export as ifc. 00-ESQ.zip

Thanks in advance for any feedback you might have

janbrouwer commented 1 year ago

I don't see the 'floating roof' in any of the viewers and checkerd I have currently installed, I will make a new release, can you test if you still see that there? Could it be an issue in eveBIM? I did fix an issue for IfcRoof used as a container. I also made a change in version 5 where unclassified objects are grouped inside the first IFC entity higher in the conponent hierarchy, this prevents models without any IFC classifications being merged into one big blob.

valleyarchitect commented 1 year ago

Hi Jan, thanks for your feedback.

The ifc exported with 3.4.7 allows for "floating" the roof in eveBIM or other viewers that have that "exploded view" function, whereas the ifc exported using 5.0.2 no longer worked... Yes, I'll gladly test a tweaked version :)

Best regards, Patrick Thomas architecte dplg T: +33(0)610045410

http://www.vivarchi.f http://www.vivarchi.frr

On Mon, 2 Oct 2023 at 09:32, Jan Brouwer @.***> wrote:

I don't see the 'floating roof' in any of the viewers and checkerd I have currently installed, I will make a new release, can you test if you still see that there? Could it be an issue in eveBIM? I did fix an issue for IfcRoof used as a container. I also made a change in version 5 where unclassified objects are grouped inside the first IFC entity higher in the conponent hierarchy, this prevents models without any IFC classifications being merged into one big blob.

— Reply to this email directly, view it on GitHub https://github.com/BIM-Tools/SketchUp-IFC-Manager/issues/51#issuecomment-1742528593, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQYBTQU4MXNPITDVFMRPRGDX5JU2TANCNFSM6AAAAAA5EEGVTM . You are receiving this because you authored the thread.Message ID: @.***>

valleyarchitect commented 1 year ago

Jan,

IFCManager-347-danseveBIM IFCManager-503 IFCManager-503-dans eveBIM

I've tried v503 and it's still doing odd things... as you will see from the sketchup screenshot, a whole bunch of "context" is hidden, yet appears in the ifc file opened in eveBIM even though i've not selected the option to export hidden geometry. Furthermore, the roof storey still doesn't lift correctly in the exploded view. As a comparison, you'll see that an ifc from the same SU model exported with version 347 of your plugin gives good results ! I have no idea why this is happening :( Thanks anyway, Best regards,

PT