Issue549 - Added the IfcGeometricRepresentationContext "Plan" in the …

[CCT-Mukund-Thakare]

…filter.

[andyward]

Thanks for this Mukund! Sorry I was replying with some suggestions on the issue. I'd not add Plan explicitly wrong according to the standards, but I would open up the filter on this fallback code to include any RepresentationContext that supports 3D, for the case where there model if not following the standards.

[CCT-Mukund-Thakare]

Hello @andyward
I get it, I should not add 'plan' because xbim.geometry.engine is only interested in 3D context type and 'plan' is reserved for 2D.
and we should search for more 3D ContextType rather than 2D ContextTypes in the model.

please check out my this comment as well.
xBimTeam/XbimEssentials#549 (comment)

If this is so can we close this PR ?

[andyward]

Yes I think the fix is probably wrong. This code is trying to find the most appropriate set of Contexts to use for 3D. Adding in Plan for a specifically wrong model doesn't make sense. Suggest we have another go. Will update on the 549 issue