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

Pull request #71 IFC error #73

Closed
larswik opened this issue Mar 19, 2021 · 8 comments · Fixed by #71
Closed

Pull request #71 IFC error #73

larswik opened this issue Mar 19, 2021 · 8 comments · Fixed by #71

Comments

@larswik
Copy link
Collaborator

larswik commented Mar 19, 2021

See #71 : I get the following error from KIT checker for the alignment file:

#56=IFCSHAPEREPRESENTATION(#55,'Axis','Curve2D',(#50))

where rule 'IFCSHAPEREPRESENTATION.CORRECTITEMSFORTYPE' evaluates to FALSE

#107=IFCSHAPEREPRESENTATION(#55,'Axis','Curve3D',(#106))

where rule 'IFCSHAPEREPRESENTATION.CORRECTITEMSFORTYPE' evaluates to FALSE

EDIT: markdown formatting

@pjanck
Copy link
Member

pjanck commented Mar 19, 2021

Perhaps this could be the reason?

@larswik
Copy link
Collaborator Author

larswik commented Mar 19, 2021

What exactly do you mean with this?

@AlexBrad1eyCT AlexBrad1eyCT linked a pull request Mar 19, 2021 that will close this issue
23 tasks
@pjanck
Copy link
Member

pjanck commented Mar 19, 2021

Well, it's not valid STEP syntax, so it could be, that the reader has problems with figuring out, what entity it is, and thus flags the shape representation as false. This is just a wild guess.

@pjanck
Copy link
Member

pjanck commented Mar 19, 2021

Ok, I see you've updated the IFC file to remove the error. Do you still get the same error, though?

@larswik
Copy link
Collaborator Author

larswik commented Mar 19, 2021

I ran the KIT checker locally on a new version of the file (which I hadn't updated here - hence the confusion). The errors refer to the file that is currently uploaded.

@larswik
Copy link
Collaborator Author

larswik commented Mar 19, 2021

I can't see what is wrong. The first refers to the horizontal alignment which refers to the IfcCompositeCurve which starts with a 2D line. The second refers to the vertical alignment which refers to an IfcGradientCurve which starts with a curve segment/IfcPolynomialCurve.

@pjanck
Copy link
Member

pjanck commented Mar 19, 2021

Then, I'm really confused. I come to the same conclusions as you.

Looking at the code below, though: there is no DERIVE: Dim parameter specified that would return Segments[0].Dim or something similar. So it makes sense, that the checker returns false.

https://github.com/bSI-InfraRoom/IFC-Specification/blob/64f1fc0e8cdf8aba82d2b99a423ddefae2f5a2ba/IFC4x3_RC3.exp#L5105-L5119

@pjanck
Copy link
Member

pjanck commented May 4, 2021

Fixed in the recent updates to the schema and checker.

@pjanck pjanck closed this as completed May 4, 2021
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

Successfully merging a pull request may close this issue.

2 participants