Jan Brouwer

Results 18 comments of Jan Brouwer

Does this mean that ConnectEdgesToWires is broken in 7.4.0? Sorry for the newbie question... I got this IfcOpenShell sample running on 7.4.0 and the only thing that fails is ConnectEdgesToWires....

I don't know if IfcOpenShell can be forced to use a specific encoding method (the shorter version saves some disk space) but "D\X4\000000F8\X0\r" to me seems a perfect representation of...

For now the main focus has been on IFC SPF to ifcJSON, roundtripping should work but it's not activated by default due to an additional dependency on pandas. Can you...

Yes, fixing that should be easy. Just: ``` pip install pandas ``` or ``` conda install pandas ```

I'm sorry, it's been a busy week! The round-trip currently only only works with "official" IFC schema versions that are recognized by ifcopenshell. The IFC5a version is the "forward looking"...

I would at any time vote for PascalCase over uppercase when strings are concatenated parts. Any software can go from PascalCase to UPPERCASE but the reverse is not possible without...

I think I made that regex, an experiment to see if it is possible to validate a positivelengthmeasure, I believe the regex validation site mentioned in the IDS docs thought...

@berndhahnebach I added a comparison for all "BuildingSMARTSpec" files for some IFC apps I have access to to the [README](https://github.com/buildingSMART/Sample-Test-Files/blob/master/IFC%204.0/BuildingSMARTSpec/README.md), it would be great if you can add more for...

> I am still not a big fan of the header, as I think that it is content that belongs in a file system, not inside a file. > >...

Proposal for exchange of IFC JSON for a complete model (for snippets, just the contents of "data"): ``` { "type": "ifcjson", "schema": "IFC4", "description": "ViewDefinition [CoordinationView]", "timeStamp": "2020-05-30T09:09:10", "preprocessorVersion": "IFC-manager...