MantisBT - Community
View Issue Details
0024954Community[OCCT] OCCT:Modeling Algorithmspublic2014-05-25 14:582017-08-04 12:11
shoogen 
bugmaster 
normalminor 
closedfixed 
Linux 64-bitDebian7.5
 
[OCCT] 7.3.0[OCCT] 6.9.0 
bugs modalg_7 bug24954_1, bug24954_2, bug24954_3, bug24954_4, bug24954_5, bug24954_6, bug24954_7, bug24954_8, bug24954_9
0024954: BRepBuilderAPI_MakeFace adds wire with aribitary orientation
When 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)
#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
No tags attached.
related to 0024255closed bugmaster Open CASCADE Failure of some test cases on OCCT vc9 win64 Release 
Issue History
2014-05-25 14:58shoogenNew Issue
2014-05-25 14:58shoogenAssigned To => ifv
2014-05-25 17:52shoogenNote Added: 0029503
2014-05-26 11:10shoogenNote Edited: 0029503bug_revision_view_page.php?bugnote_id=29503#r7536
2014-09-12 14:04abvTarget Version => 6.8.0
2014-10-06 14:34ifvRelationship addedrelated to 0024255
2014-10-29 06:56abvTarget Version6.8.0 => 7.1.0
2016-10-28 11:54msvTarget Version7.1.0 => 7.2.0
2017-07-21 11:34msvTarget Version7.2.0 => 7.3.0
2017-08-01 14:45gitNote Added: 0069023
2017-08-01 14:47apvAssigned Toifv => apv
2017-08-01 14:47apvStatusnew => feedback
2017-08-01 14:49apvTest 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:49apvNote Added: 0069024
2017-08-01 14:49apvAssigned Toapv => bugmaster
2017-08-01 14:49apvStatusfeedback => tested
2017-08-01 14:49apvFixed in Version => 6.9.0
2017-08-03 18:13gitNote Added: 0069100
2017-08-04 12:11apvStatustested => closed
2017-08-04 12:11apvResolutionopen => fixed

Notes
(0029503)
shoogen   
2014-05-25 17:52   
(edited on: 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 )

(0069023)
git   
2017-08-01 14:45   
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
(0069024)
apv   
2017-08-01 14:49   
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.
(0069100)
git   
2017-08-03 18:13   
Branch CR24954 has been deleted by mkv.

SHA-1: 208e09500f40fa46dfb1b053bfe2bfb69ecdc117