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

Open
CEXT-Dan opened this issue Jul 7, 2024 · 34 comments
Open

Wrap BrxBim classes #90

CEXT-Dan opened this issue Jul 7, 2024 · 34 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
@schoeller
Copy link
Contributor

Advise is requested:

import traceback
from pyrx_imp import Rx, Ge, Gs, Gi, Db, Ap, Ed, Bim

import wx
import os, pathlib

def openIfcSideDrawing(path, opts):
    print("\nProcessing {} ".format(path))
    db = Db.Database(False, True)
    Bim.IfcImportOptions.importIfcFile(db, path, opts)
    path = str(path[:-3]) + "dwg"
    print(path)
    db.saveAs("c://temp/test.dwg")
    db.closeInput(True)
            
def PyRxCmd_doit1():
    try:
        ifcext = ".ifc".casefold()
        opts = Bim.IfcImportOptions()

        dlg = wx.DirDialog(None, "Choose input directory","",
                wx.DD_DEFAULT_STYLE | wx.DD_DIR_MUST_EXIST)
    
        if dlg.ShowModal() != wx.ID_OK:
            print("You Cancelled The Dialog!")
            return

        for fname in next(os.walk(dlg.GetPath()), (None, None, []))[2]:
            ext = pathlib.Path(fname).suffix.casefold()
            if ext != ifcext:
                continue
            fpath = '{}\\{}'.format(dlg.GetPath(),fname)
            print(fpath)
            openIfcSideDrawing(fpath, opts)
    except Exception as err:
        traceback.print_exception(err)

leads to

Processing C:\Users\Sebastian Schoeller\Downloads\Ifc\13-02_40-00_IWP.ifc 
C:\Users\Sebastian Schoeller\Downloads\Ifc\13-02_40-00_IWP.dwg
Traceback (most recent call last):
  File "C:\Users\Sebastian Schoeller\Nextcloud\10000-Beruf\02-Software\PyRx\ConvertIfc.py", line 34, in PyRxCmd_doit1
    openIfcSideDrawing(fpath, opts)
  File "C:\Users\Sebastian Schoeller\Nextcloud\10000-Beruf\02-Software\PyRx\ConvertIfc.py", line 13, in openIfcSideDrawing
    db.saveAs("c://temp/test.dwg")
PyDb.ErrorStatusException: Exception!(eFileAccessErr), function saveAs1, Line 2150, File PyDbDatabase.cpp:

@CEXT-Dan
Copy link
Owner Author

this works for me, see my comment
also, make sure c:/temp/test.dwg is not opened already

import traceback
from pyrx_imp import Rx, Ge, Gs, Gi, Db, Ap, Ed, Bim

import wx
import os, pathlib

def openIfcSideDrawing(path, opts):
    print("\nProcessing {} ".format(path))
    
    # the ctor is going to change because in this case you are creating 
    # a new drawing which also needs a new document 
    db = Db.Database(True, False) 
    Bim.IfcImportOptions.importIfcFile(db, path, opts)
    path = str(path[:-3]) + "dwg"
    print(path)
    db.saveAs("c:/temp/test.dwg")
    #not needed, this is for reading exisiting drawings
    #db.closeInput(True)
            
def PyRxCmd_doit1():
    try:
        ifcext = ".ifc".casefold()
        opts = Bim.IfcImportOptions()

        dlg = wx.DirDialog(None, "Choose input directory","",
                wx.DD_DEFAULT_STYLE | wx.DD_DIR_MUST_EXIST)
    
        if dlg.ShowModal() != wx.ID_OK:
            print("You Cancelled The Dialog!")
            return

        for fname in next(os.walk(dlg.GetPath()), (None, None, []))[2]:
            ext = pathlib.Path(fname).suffix.casefold()
            if ext != ifcext:
                continue
            fpath = '{}\\{}'.format(dlg.GetPath(),fname)
            print(fpath)
            openIfcSideDrawing(fpath, opts)
    except Exception as err:
        traceback.print_exception(err)

db

@CEXT-Dan
Copy link
Owner Author

oh! also db.saveAs("c:/temp/test.dwg") is in a loop, so if you have more than one .ICF file if will fail because c:/temp/test.dwg is already there

@schoeller
Copy link
Contributor

Thx. Blindfolded. This works for me now:

def openSaveIfcSideDrawing(path, opts):
    print("\nProcessing {} ".format(path))
    db = Db.Database(True, False)
    Bim.IfcImportOptions.importIfcFile(db, path, opts)
    path = str(path[:-3]) + "dwg"
    db.saveAs(path)
    print("\nSaved to {} ".format(path))
    db.closeInput(True)

@CEXT-Dan
Copy link
Owner Author

I’m not sure what to do with this task. The import reactor, (which seems like the most important feature? ) does not work in V24 or V25. I think the class layout is mostly done. I’ll reopen this so it can be worked on.
Maybe there’s some other important features, I don’t really understand BIM

@CEXT-Dan CEXT-Dan reopened this Nov 21, 2024
@schoeller
Copy link
Contributor

Started test scripting the sample from BRX25.1.05.0\samples\brxBimSample\BimClassification.cpp. Noticed that Bim.BimStory.setDescription was non-existant. Compared BRX25.1.05.0\inc\BrxSpecific\bim\BuildingElements.h to PyBrxBimObject.h and have openend a pull request for what I thought to be useful with focus on BimStory.

When doing

ourBuilding = Bim.BimBuilding.createNewBuilding(db, "TowerNorth")
ourBuilding.setDescription("the north side tower building")
floor_1 = Bim.BimStory.createNewStory(db, "TowerNorth", "Floor 1")

all nice, but

floor_2 = Bim.BimStory.createStory("second floor", ourBuilding)

throws

    floor_2 = Bim.BimStory.createStory("second floor", ourBuilding)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Boost.Python.ArgumentError: Python argument types in
    BimStory.createStory(str, BimBuilding)
did not match C++ signature:
    createStory(class PyBrxBimStory {lvalue}, class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> >, class PyBrxBimBuilding)

@schoeller
Copy link
Contributor

When trying as per SDK

    // assign floors & building to "hull entities"
    AcDbObjectId idBuilding;
    AcDbObjectIdArray idsForBuilding;
    bimRes = ourBuilding.assignedObjects(idsForBuilding, pDb);
    if (idsForBuilding.isEmpty())
    {
        idBuilding = createBoxSolid(100, 35, 20, AcGeVector3d(0,0,10.0), 1);
        bimRes = ourBuilding.assignToEntity(idBuilding);
    }

in the fashion

        db = Db.curDb()
        model = Db.BlockTableRecord(Db.curDb().modelSpaceId(), Db.OpenMode.ForRead)

        ourBuilding = Bim.BimBuilding.createNewBuilding(db, "TowerNorth")
        ourBuilding.setDescription("the north side tower building")

        if not ourBuilding.assignedObjects(db):
            print("creating building elements")
            box =  Db.Solid3d()
            box.createBox(100.0,35.0,20.0) # createBoxSolid(100, 35, 20, AcGeVector3d(0,0,10.0), 1); add vector
            model.upgradeOpen()
            model.appendAcDbEntity(box)
            ourBuilding.assignToEntity(box.id())
            model.downgradeOpen()

I see no properties attached to the solid.

@CEXT-Dan
Copy link
Owner Author

Started test scripting the sample from BRX25.1.05.0\samples\brxBimSample\BimClassification.cpp. Noticed that Bim.BimStory.setDescription was non-existant. Compared BRX25.1.05.0\inc\BrxSpecific\bim\BuildingElements.h to PyBrxBimObject.h and have openend a pull request for what I thought to be useful with focus on BimStory.

When doing

ourBuilding = Bim.BimBuilding.createNewBuilding(db, "TowerNorth")
ourBuilding.setDescription("the north side tower building")
floor_1 = Bim.BimStory.createNewStory(db, "TowerNorth", "Floor 1")

all nice, but

floor_2 = Bim.BimStory.createStory("second floor", ourBuilding)

throws

    floor_2 = Bim.BimStory.createStory("second floor", ourBuilding)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Boost.Python.ArgumentError: Python argument types in
    BimStory.createStory(str, BimBuilding)
did not match C++ signature:
    createStory(class PyBrxBimStory {lvalue}, class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> >, class PyBrxBimBuilding)

createStory is not static

def PyRxCmd_doit():
    try:
        db = Db.curDb()
        ourBuilding = Bim.BimBuilding.createNewBuilding(db, "TowerNorth")
        ourBuilding.setDescription("the north side tower building")
        floor_1 = Bim.BimStory.createNewStory(db, "TowerNorth", "Floor 1")

        floor_2 = Bim.BimStory()
        floor_2.createStory("second floor", ourBuilding)

        print(
            ourBuilding.description(),
            floor_2.getBuilding().description() == ourBuilding.description(),
        )

    except Exception as err:
        traceback.print_exception(err)

@CEXT-Dan
Copy link
Owner Author

if not ourBuilding.assignedObjects(db):

looks like a bug in BricsCAD. the function succeeds;

from pyrx_imp import Rx, Ge, Gs, Gi, Db, Ap, Ed, Bim
import traceback

def PyRxCmd_doit():
    try:
        db = Db.curDb()
        ourBuilding = Bim.BimBuilding.createNewBuilding(db, "TowerNorth")
        ourBuilding.setDescription("the north side tower building")
        ourBuilding.setName("TowerNorth")
        ourBuilding.setLongName("TowerNorth")

        if not ourBuilding.assignedObjects(db):
            print("creating building elements")
            box =  Db.Solid3d()
            box.createBox(100.0,35.0,20.0)
            box.transformBy(Ge.Matrix3d.translation(Ge.Vector3d(0,0,10.0)))
            bid = db.addToModelspace(box)
            ourBuilding.assignToEntity(bid)
    except Exception as err:
        traceback.print_exception(err)

setdesc

BTW, db.addToModelspace is a shortcut for having to deal with model

CEXT-Dan added a commit that referenced this issue Nov 26, 2024
add BimClassification get, set name
add BimClassification get, set description(
@schoeller
Copy link
Contributor

schoeller commented Nov 26, 2024

Comparing BuildingElements.h to PyBrxBimObject.h for PyBrxBimClassification. Have seen that there are some double entries like below

static BimApi::ResultStatus classifyAs(const AcDbObjectId& id, const BimApi::BimElementType objectType);
static BimApi::ResultStatus classifyAs(const AcDbObjectId& id, const AcString& typeName, bool localName = false);    

@CEXT-Dan
Copy link
Owner Author

CEXT-Dan commented Nov 26, 2024

Comparing BuildingElements.h to PyBrxBimObject.h for PyBrxBimClassification. Have seen that there are some double entries like below

static BimApi::ResultStatus classifyAs(const AcDbObjectId& id, const BimApi::BimElementType objectType);
static BimApi::ResultStatus classifyAs(const AcDbObjectId& id, const AcString& typeName, bool localName = false);    

Unless we know for certain that one method is depreciated, we have to do both,
we don’t know if the user will run into a situation where the overload is required

edit: Just to add, of course, you can only do the functions you need to get your project working, maybe leave a TODO comment that the class is incomplete

CEXT-Dan added a commit that referenced this issue Nov 27, 2024
CEXT-Dan added a commit that referenced this issue Nov 27, 2024
Updating PyBrxBimBuildingWrapper [#90'
: )
CEXT-Dan added a commit that referenced this issue Nov 27, 2024
@schoeller
Copy link
Contributor

Have no idea what I am doing. How would this translate in BuildingElements.h?

class BRX_IMPORTEXPORT BimClassification
...
static BimApi::ResultStatus getClassification(AcString& typeName, const AcDbObjectId& id, bool localName = false);
...

@CEXT-Dan
Copy link
Owner Author

Have no idea what I am doing. How would this translate in BuildingElements.h?

class BRX_IMPORTEXPORT BimClassification
...
static BimApi::ResultStatus getClassification(AcString& typeName, const AcDbObjectId& id, bool localName = false);
...

it's a hard class, I committed and update,I will finish it when I am back later today

@CEXT-Dan
Copy link
Owner Author

BimClassification should be ready, an insane amount of overloads. I took inspiration from Bim .NET where names clashed

@schoeller
Copy link
Contributor

schoeller commented Nov 29, 2024

Trying to resume the status and where further contributions are required. Double-checking is welcome.

  • BimObjectType
  • BimElementType
  • BrxBimPolicies
  • BrxBimObject
  • BrxBimSpatialLocation
  • BrxBimStory
  • BrxBimBuilding
  • BrxBimRoom
  • BrxBimSpace
  • BrxBimMaterial -> omitted as not being relevant for classification
  • InformationalAssets -> currently unknown
  • BimAssets -> currently unknown
  • BrxBimPly -> omitted as not being relevant for classification
  • BrxBimComposition => may be considered later
  • BrxBimProfile -> omitted as not being relevant for classification
  • BrxBimLinearGeometry -> omitted as not being relevant for classification
  • BimClassification
  • BimNameSpaces
  • BrxBimAttributeSet => may be considered later
  • BrxBimDialogs -> omitted as being relevant for GUI only
  • BimLibraryInfo -> omitted as not being relevant for classification
  • BrxBimImplT -> currently unknown
  • BrxBimImplBase -> currently unknow
  • BimPropertySets => may be considered later

@CEXT-Dan
Copy link
Owner Author

Looks right.

Just to add:
Also everything in IfcExportReactor.h is missing, I stopped because the import reactor seems not to work in BRX

BimPropertySets looks quite hard to wrap

listAllPsetProperties “std::map<AcString, std::map<AcString, std::map<AcString, AcValue>>>:

IMHO the API isn’t well thought out, Bricsys had to alter the API to be more usable from .NET
If you have everything you need, I will work on missing items over the coming weeks

@schoeller
Copy link
Contributor

IMHO the API isn’t well thought out, Bricsys had to alter the API to be more usable from .NET If you have everything you need, I will work on missing items over the coming weeks

I should be very fine. Thanks a ton for letting me participate. Am learning from your coding and corrected contributions.

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