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

Wrap BrxBim classes #90

Closed
CEXT-Dan opened this issue Jul 7, 2024 · 17 comments
Closed

Wrap BrxBim classes #90

CEXT-Dan opened this issue Jul 7, 2024 · 17 comments
Labels
enhancement New feature or request

Comments

@CEXT-Dan
Copy link
Owner

CEXT-Dan commented Jul 7, 2024

@CEXT-Dan hope you're well. Could use functionality to import 1) IFC, 2) classify entities (for example by layer name) and 3) export to IFC. I believe it's all BCAD verticals. I am aware that there may be other priorities. Whenever you find the time I would be grateful to be able to script anything.

  1. https://help.bricsys.com/en-us/document/command-reference/i/import-command?version=V24&id=165079082118
  2. https://help.bricsys.com/en-us/document/bricscad-bim/building-data/bim-classify?version=V24&id=165079158884
  3. https://help.bricsys.com/en-us/document/command-reference/i/ifcexport-command?version=V24&id=165079081611

Have in mind that you're about to travel for longer. Wish you safe journey.

Originally posted by @schoeller in #5 (comment)

@CEXT-Dan CEXT-Dan self-assigned this Jul 7, 2024
@CEXT-Dan CEXT-Dan added the enhancement New feature or request label Jul 7, 2024
@CEXT-Dan
Copy link
Owner Author

CEXT-Dan commented Jul 7, 2024

In progress, est two-three weeks

@schoeller
Copy link
Contributor

Saw that you're back online. Hope you're well. Please signal whenever I becomes reasonable to attempt scripting BRX24.2.03.0\samples\brxBimSample\BimIfcImport.cpp

@CEXT-Dan
Copy link
Owner Author

CEXT-Dan commented Sep 3, 2024

I’m going to defer this until V25 is released, reason is, there appears API changes between beta versions. I find myself having to redo stuff.

@CEXT-Dan
Copy link
Owner Author

CEXT-Dan commented Sep 5, 2024

Saw that you're back online. Hope you're well. Please signal whenever I becomes reasonable to attempt scripting BRX24.2.03.0\samples\brxBimSample\BimIfcImport.cpp

Hi, do you have any sample data I can use to test? I'll try to push hard on this next week. Admittedly, this wrap is hard for me, I've never worked with BIM.

@schoeller
Copy link
Contributor

Saw that you're back online. Hope you're well. Please signal whenever I becomes reasonable to attempt scripting BRX24.2.03.0\samples\brxBimSample\BimIfcImport.cpp

Hi, do you have any sample data I can use to test? I'll try to push hard on this next week. Admittedly, this wrap is hard for me, I've never worked with BIM.

Hi as well. I just tried sample data from BuildingSMART. Works as manual import into v24. Hope distraction from vanilla work does not cause too many headaches.

From my current understanding the BCAD BIM (in this case) consists of two parts:

1#Import of geometries into ACIS objects from IFC 2x3 up to 4.1.
grafik

2#Classification of existing objects through a defined tag-catalogue.
grafik

In a current project it would be a time-saver to mass-convert incoming IFC-files to DWG to assemble a collision model (1#). Second goal would be to sync object classification <-> layer structure.

Best wishes.

Seb

@schoeller
Copy link
Contributor

@CEXT-Dan I saw the code snipped from your work. I have tried to load it with the file path correctly set. It messages as follows:

onIfcProduct 94257
<PyBrxBim.IfcImportContext object at 0x00000000030F1F40> False
True

I do not see any objects imported though. I cannot get around figuring out the usage of Bim.IfcImportReactor. Is it already possible to sideload (convert) an IFC to DWG and how would I do that? Best Seb

@CEXT-Dan
Copy link
Owner Author

Hi, it's not ready yet.

@CEXT-Dan
Copy link
Owner Author

I can import the sample from building smart on a local compile. However, I’m having trouble in that onIfcProductImported is never called. It’s not called in the BRX sample either. onIfcProduct is called, however, matrix = context.getLocalPlacement(entity) always returns an identity, This is also the same with the BRX sample.
I’m not sure if this is by design or if BRX has issues. I’ll work this week on filling in the gaps for importing, I expect there will be issues

@CEXT-Dan
Copy link
Owner Author

SR 184312 asks for help

@schoeller
Copy link
Contributor

I can import the sample from building smart on a local compile. However, I’m having trouble in that onIfcProductImported is never called. It’s not called in the BRX sample either. onIfcProduct is called, however, matrix = context.getLocalPlacement(entity) always returns an identity, This is also the same with the BRX sample. I’m not sure if this is by design or if BRX has issues. I’ll work this week on filling in the gaps for importing, I expect there will be issues

I have seen the following in the SDK as well as samples\brxBimSample\BimIfcImport.cpp. Everything else is out of my league

grafik

@CEXT-Dan
Copy link
Owner Author

I had set new guid in
https://github.com/CEXT-Dan/PyRx/blob/main/PySamples/PyBrxBim/importTest.py

onStart, onIfcProduct, and beforeCompletion are called correctly. My guess is that onIfcProductImported has lost its “virtual” status due to some internal change. I still have lots of work on other items in BIM while I wait on the SR… I’ll make a release Thursday as a progress build

@schoeller
Copy link
Contributor

I had set new guid in https://github.com/CEXT-Dan/PyRx/blob/main/PySamples/PyBrxBim/importTest.py

onStart, onIfcProduct, and beforeCompletion are called correctly. My guess is that onIfcProductImported has lost its “virtual” status due to some internal change. I still have lots of work on other items in BIM while I wait on the SR… I’ll make a release Thursday as a progress build

Sorry to have bothered you with this workload.

@schoeller
Copy link
Contributor

Is it somehow possible to fetch properties from Mechanical info (1#) (for example through dump functionality) and copy it into a custom BIM properties set (2#)?
Just curious to save us tons of hours.

1#

2#

@CEXT-Dan
Copy link
Owner Author

Not sure about that off the top of my head
I noticed the mechanical sample that has access to properties. I know Bricsys has a cool properties class for vanilla BricsCAD, I have not wrapped either yet

@schoeller
Copy link
Contributor

Not sure about that off the top of my head I noticed the mechanical sample that has access to properties. I know Bricsys has a cool properties class for vanilla BricsCAD, I have not wrapped either yet

@CEXT-Dan: Thx for the feedback. We will try ourselves with LISP standard tools for the mo. Best of health.

@CEXT-Dan
Copy link
Owner Author

Bricsys was able to reproduce the import reactor issue, I don’t think they will be working on a V24 fix, since V25 is just around the corner. I’ll continue working on Bim wrappers, albeit with a lower priority

@CEXT-Dan CEXT-Dan removed their assignment Nov 7, 2024
@CEXT-Dan
Copy link
Owner Author

CEXT-Dan commented Nov 7, 2024

Seems Bricsys does not want to support me on this project. Closing until something changes

@CEXT-Dan CEXT-Dan closed this as completed Nov 7, 2024
@CEXT-Dan CEXT-Dan closed this as not planned Won't fix, can't repro, duplicate, stale Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants