View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0024954CommunityOCCT:Modeling Algorithmspublic2014-05-25 14:582017-08-04 12:11
Reportershoogen Assigned Tobugmaster  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
PlatformLinux 64-bitOSDebian 
Target Version7.3.0Fixed in Version6.9.0 
Summary0024954: BRepBuilderAPI_MakeFace adds wire with aribitary orientation
DescriptionWhen creating simple faces (triagles, rectangles) I would assume that orientation is kept straight forward. This works fine for edges and vertices.
But when i call BRepBuilderAPI_MakeFace with a single wire, the orientation of the wire is sometimes reversed, which results in an face normal which has the wrong direction compared to the cross product of the edges used (and the face normal intended for the surface creation)
Steps To Reproduce#Example for correct face normal
polyline pf1 5.24 1.49 0.69 5.93 3.99 2.61 -0.21 2.53 3.15 5.24 1.49 0.69
wire wf1 pf1
mkplane ff1 wf1
#ff1 has the wire in face Forward:
#TShape # 1 : FACE 1111000
# +2

#Example for incorrect face normal
polyline pr1 9.08 8.62 3.75 1.87 3.04 9. 4.26 1.37 -0.08 9.08 8.62 3.75
wire wr1 pr1
mkplane fr1 wr1
#fr1 has the wire in face Reversed:
#TShape # 1 : FACE 1101000
# -2

#further polylines that lead to wrong face normals
polyline p3 -1.15 3.42 1.95 4.54 3.13 1.36 2.97 -3.14 2.08 -1.15 3.42 1.95
polyline p4 -4.62 0.1 0.75 -4.12 4.64 -2.91 -1.55 -1.07 4.48 -4.62 0.1 0.75
polyline p5 -2.92 -4.7 -2.87 4.73 1.75 1.76 3.14 -1.23 2.74 -2.92 -4.7 -2.87
polyline p6 -4.05 3.84 0.17 4.22 3.55 3.19 -1.32 -1.44 -1.57 -4.05 3.84 0.17
polyline p7 -2.36 -1.62 0.17 3.26 -1.05 -3.8 2.75 -4.01 4.95 -2.36 -1.62 0.17
polyline p8 4.72 2.03 -2.52 -4.78 1.89 -3.82 -1.42 4.56 1.55 4.72 2.03 -2.52
polyline p9 2.02 -1.4 -3.75 -2.93 -1.87 1.72 -5. -0.18 0.39 2.02 -1.4 -3.75
polyline p10 1.21 0.16 4.2 3.12 4.16 0.93 -2.33 -2.55 2.41 1.21 0.16 4.2
polyline p11 3.99 -3.09 -1.37 2.33 -3.18 2.35 -3.06 0.9 -3.08 3.99 -3.09 -1.37
polyline p12 0.29 4.98 3.68 2.13 3.02 2.84 1.61 0.34 4.46 0.29 4.98 3.68
TagsNo tags attached.
Test case numberbugs modalg_7 bug24954_1, bug24954_2, bug24954_3, bug24954_4, bug24954_5, bug24954_6, bug24954_7, bug24954_8, bug24954_9

Relationships

related to 0024255 closedbugmaster Open CASCADE Failure of some test cases on OCCT vc9 win64 Release 

Activities

shoogen

2014-05-25 17:52

reporter   ~0029503

Last edited: 2014-05-26 11:10

 I understand that there is a relation between wire and surface. But from that point of view the surface is chosen badly (by BRepLib_FindSurface )

git

2017-08-01 14:45

administrator   ~0069023

Branch CR24954 has been created by apv.

SHA-1: 208e09500f40fa46dfb1b053bfe2bfb69ecdc117


Detailed log of new commits:

Author: apv
Date: Tue Aug 1 14:45:03 2017 +0300

    Test for 0024954: BRepBuilderAPI_MakeFace adds wire with aribitary orientation

apv

2017-08-01 14:49

tester   ~0069024

Dear BugMaster,

Problem described in issue is not reproduced on current state of OCCT.
Problem has been solved in OCCT6.9.0.

Please integrate branch CR24954 into the occt git-repository master.

git

2017-08-03 18:13

administrator   ~0069100

Branch CR24954 has been deleted by mkv.

SHA-1: 208e09500f40fa46dfb1b053bfe2bfb69ecdc117

Issue History

Date Modified Username Field Change
2014-05-25 14:58 shoogen New Issue
2014-05-25 14:58 shoogen Assigned To => ifv
2014-05-25 17:52 shoogen Note Added: 0029503
2014-05-26 11:10 shoogen Note Edited: 0029503
2014-09-12 14:04 abv Target Version => 6.8.0
2014-10-06 14:34 ifv Relationship added related to 0024255
2014-10-29 06:56 abv Target Version 6.8.0 => 7.1.0
2016-10-28 11:54 msv Target Version 7.1.0 => 7.2.0
2017-07-21 11:34 msv Target Version 7.2.0 => 7.3.0
2017-08-01 14:45 git Note Added: 0069023
2017-08-01 14:47 apv Assigned To ifv => apv
2017-08-01 14:47 apv Status new => feedback
2017-08-01 14:49 apv Test case number => bugs modalg_7 bug24954_1, bug24954_2, bug24954_3, bug24954_4, bug24954_5, bug24954_6, bug24954_7, bug24954_8, bug24954_9
2017-08-01 14:49 apv Note Added: 0069024
2017-08-01 14:49 apv Assigned To apv => bugmaster
2017-08-01 14:49 apv Status feedback => tested
2017-08-01 14:49 apv Fixed in Version => 6.9.0
2017-08-03 18:13 git Note Added: 0069100
2017-08-04 12:11 apv Status tested => closed
2017-08-04 12:11 apv Resolution open => fixed