Door-representation-improvements-ak

Elements.Serialization.IFC/src/IFCToHypar/Converters/FromIfcDoorConverter.cs

@@ -34,16 +34,18 @@ public GeometricElement ConvertToElement(IfcProduct ifcProduct, RepresentationDa
 
             // TODO: Implement during the connections establishment.
             //var wall = GetWallFromDoor(ifcDoor, allWalls);
-            var doorWidth = (IfcLengthMeasure) ifcDoor.OverallWidth;
-            var doorHeight = (IfcLengthMeasure) ifcDoor.OverallHeight;
+            var doorWidth = (IfcLengthMeasure)ifcDoor.OverallWidth;
+            var doorHeight = (IfcLengthMeasure)ifcDoor.OverallHeight;
 
             var result = new Door(doorWidth,
                                   doorHeight,
+                                  Door.DOOR_THICKNESS,
                                   openingSide,
                                   openingType,
                                   repData.Transform,
                                   repData.Material,
                                   new Representation(repData.SolidOperations),
+                                  false,
                                   IfcGuid.FromIfcGUID(ifcDoor.GlobalId),
                                   ifcDoor.Name
                                   );

Elements.Serialization.IFC/test/IFCTests.cs

@@ -8,6 +8,7 @@
 using System.Collections.Generic;
 using Elements.Geometry.Profiles;
 using System.Linq;
+using System.Xml.Linq;
 
 namespace Elements.IFC.Tests
 {
@@ -28,7 +29,7 @@
         // [InlineData("rac_sample", "../../../models/IFC4/rac_advanced_sample_project.ifc")]
         // [InlineData("rme_sample", "../../../models/IFC4/rme_advanced_sample_project.ifc")]
         // [InlineData("rst_sample", "../../../models/IFC4/rst_advanced_sample_project.ifc")]
-        [InlineData("AC-20-Smiley-West-10-Bldg", "../../../models/IFC4/AC-20-Smiley-West-10-Bldg.ifc", 1972, 120, 539, 270, 9, 140, 10, 2)]
+        [InlineData("AC-20-Smiley-West-10-Bldg", "../../../models/IFC4/AC-20-Smiley-West-10-Bldg.ifc", 1963, 120, 530, 270, 9, 140, 10, 2)]
         // TODO: Some walls are extracted incorrectly and intersecting the roof. It happens because
         // IfcBooleanClippingResultParser doesn't handle the boolean clipping operation.
         // In order to fix it surface support is required.
@@ -38,7 +39,7 @@
         // TODO: The entrance door has an incorrect representation. It happens because during
         // the UpdateRepresentation the default representation of a door is created instead of
         // the extracted one.
-        [InlineData("AC20-Institute-Var-2", "../../../models/IFC4/AC20-Institute-Var-2.ifc", 1517, 5, 577, 121, 7, 82, 0, 21)]
+        [InlineData("AC20-Institute-Var-2", "../../../models/IFC4/AC20-Institute-Var-2.ifc", 1506, 5, 570, 121, 7, 82, 0, 21)]
         // [InlineData("20160125WestRiverSide Hospital - IFC4-Autodesk_Hospital_Sprinkle", "../../../models/IFC4/20160125WestRiverSide Hospital - IFC4-Autodesk_Hospital_Sprinkle.ifc")]
         public void FromIFC4(string name,
                          string ifcPath,
@@ -137,15 +138,18 @@
             var wall1 = new StandardWall(wallLine1, 0.2, 3, name: "Wall1");
             var wall2 = new StandardWall(wallLine2, 0.2, 2, name: "Wall2");
 
-            model.AddElement(wall1);
-            model.AddElement(wall2);
+            var door1 = new Door(wallLine1, 0.5, 1.5, 2.0, Door.DOOR_DEFAULT_THICKNESS, DoorOpeningSide.LeftHand, DoorOpeningType.DoubleSwing);
+            var door2 = new Door(wallLine2, 0.5, 1.5, 1.8, Door.DOOR_DEFAULT_THICKNESS, DoorOpeningSide.LeftHand, DoorOpeningType.DoubleSwing);
 
-            var door1 = new Door(wall1, wallLine1, 0.5, 1.5, 2.0, DoorOpeningSide.LeftHand, DoorOpeningType.DoubleSwing);
-            var door2 = new Door(wall2, wallLine2, 0.5, 1.5, 1.8, DoorOpeningSide.LeftHand, DoorOpeningType.DoubleSwing);
+            wall1.AddDoorOpening(door1);
+            wall2.AddDoorOpening(door2);
 
+            model.AddElement(wall1);
+            model.AddElement(wall2);
             model.AddElement(door1);
             model.AddElement(door2);
 
+            model.ToJson(ConstructJsonPath("IfcDoor"));
             model.ToIFC(ConstructIfcPath("IfcDoor"));
         }
 

Elements/Elements.sln

@@ -0,0 +1,31 @@
+
+Microsoft Visual Studio Solution File, Format Version 12.00
+# Visual Studio Version 17
+VisualStudioVersion = 17.5.002.0
+MinimumVisualStudioVersion = 10.0.40219.1
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Elements", "src\Elements.csproj", "{29FD57A4-88A8-4600-900E-839FDDDC2A28}"
+EndProject
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Elements.Tests", "test\Elements.Tests.csproj", "{ECCEFB6A-DF33-429B-950D-8082EA731320}"
+EndProject
+Global
+	GlobalSection(SolutionConfigurationPlatforms) = preSolution
+		Debug|Any CPU = Debug|Any CPU
+		Release|Any CPU = Release|Any CPU
+	EndGlobalSection
+	GlobalSection(ProjectConfigurationPlatforms) = postSolution
+		{29FD57A4-88A8-4600-900E-839FDDDC2A28}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{29FD57A4-88A8-4600-900E-839FDDDC2A28}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{29FD57A4-88A8-4600-900E-839FDDDC2A28}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{29FD57A4-88A8-4600-900E-839FDDDC2A28}.Release|Any CPU.Build.0 = Release|Any CPU
+		{ECCEFB6A-DF33-429B-950D-8082EA731320}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{ECCEFB6A-DF33-429B-950D-8082EA731320}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{ECCEFB6A-DF33-429B-950D-8082EA731320}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{ECCEFB6A-DF33-429B-950D-8082EA731320}.Release|Any CPU.Build.0 = Release|Any CPU
+	EndGlobalSection
+	GlobalSection(SolutionProperties) = preSolution
+		HideSolutionNode = FALSE
+	EndGlobalSection
+	GlobalSection(ExtensibilityGlobals) = postSolution
+		SolutionGuid = {5B1BA77B-C0FC-4EBD-B7B9-61E5C244207E}
+	EndGlobalSection
+EndGlobal