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

Exception details #17

Closed
Judithlourde opened this issue Feb 28, 2024 · 6 comments
Closed

Exception details #17

Judithlourde opened this issue Feb 28, 2024 · 6 comments

Comments

@Judithlourde
Copy link

Hi, I tried the python-mvdxml it works well. I have one question about the result
ConceptRoot: IfcBuilding
Concept: Spatial Decomposition
EXCEPTION: 'rule' object has no attribute 'parent'
What is the meaning of EXCEPTION?
Thanks for the answer:)

@aothms
Copy link
Member

aothms commented Feb 28, 2024

I ran into that as well recently. See this commit 00e9f56 that should fix it

@Judithlourde
Copy link
Author

Hi Thomas,
Could you please explain the result when i validate this mvdXml file with ifc file MinimalWindow.ifc

Getting this result:

ConceptRoot: IfcDoor
Concept: SingleValueProperty
0 out of 0 instances failed the check


ConceptRoot: IfcWindow
Concept: SingleValueProperty
[VALID] #13=IfcWindow('270QYpgSXDE99ViyemAGdV',#2,'Window #1',$,$,$,$,$,$,$,$,$,$)
0 out of 1 instances failed the check


RESULTS OVERVIEW
Total number of tests: 2
Total number of executed tests: 2
Total number of failed tests: 0

In the minimalWindow file there is one window do not have the fireRating property but the result shows it is valid.

@Judithlourde
Copy link
Author

Judithlourde commented Feb 28, 2024

Github does not accept mvdxml extensions.
FireRatingDemo.txt

@aothms
Copy link
Member

aothms commented Feb 28, 2024

MinimalWindow.ifc

Can you share that?

@Judithlourde
Copy link
Author

Sure!
MinimalWindow.txt

@Judithlourde Judithlourde changed the title Exeption details Exception details Mar 5, 2024
@Judithlourde
Copy link
Author

Apologies for the confusion earlier. I've reviewed the mvdXML file and found an error, which I've now corrected. I appreciate your patience and understanding. Thank you! The corrected mvdXML file is here.
FireRatingDemo (3).txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants