0023832: sprops and triarea commands return incorrect value of a mass for shape of revolution

ID	Project	Category	View Status	Date Submitted	Last Update

0023832	Open CASCADE	OCCT:Modeling Data	public	2013-03-15 18:14	2013-04-29 15:24

Reporter	oan	Assigned To	ika
Priority	normal	Severity	minor
Status	closed	Resolution	fixed
Target Version	6.6.0	Fixed in Version	6.6.0

Summary	0023832: sprops and triarea commands return incorrect value of a mass for shape of revolution
Description	sprops and triarea commands return incorrect value of a mass for shape of revolution. Both of these commands use the BRepGProp::SurfaceProperties and the GProp_GProps for definition of mass of a given shape, but the result value is much greater that expected. Exploding the result shapes on faces gives that incorrect value is calculated for initial face used for forming the whole shape. At the same time the similar face closing the shape is processed correctly. for test case bugs modalg_2 bug22818_1: triarea result # triangles - 671.7785430027094 # gemetrical - 823.38488345995472 explode result f donly result_4 result_3 sprops result_4 # Mass : 50 sprops result_3 # Mass : 200
Steps To Reproduce	bugs modalg_2 bug22818_1 bug22818_2
Tags	No tags attached.

Test case number	bugs modalg_2(007) bug22818_1, bug22818_2

ika 2013-03-19 17:05 developer ~0023803	Dear OAN, the problem is not in sprops and triarea commands. In scripts bugs modalg_2 bug22818_1 bug22818_2 faces are created incorrectly: mkface f1 pln add w1 f1 should be: mkface f1 pln w1 otherwise the value of mass calculate by non-relevant function.

oan 2013-03-19 18:06 developer ~0023805 Last edited: 2013-03-19 18:07	Dear IKA, I suppose that this way is possible for face creation, because: - usage of mkface is described as "mkface facename surfacename [ufirst ulast vfirst vlast] [wire[norient]] that means a wire is not an obligatory component; - checkshape command doesn't indicate any problem; - incmesh doesn't indicate any problem too and computes mesh correctly.

ika 2013-03-21 15:47 developer ~0023833	Creation faces in scripts bugs modalg_2 bug22818_1 and bug22818_2 was changed to right way. Branch CR23832 is ready to be reviewed. Dear GKA, Please review.

~~gka~~ 2013-03-21 15:51 developer ~0023834	Branch CR23832 is ready to be tested

~~mkv~~ 2013-03-22 20:52 tester ~0023876	Dear BugMaster, Test cases bugs modalg_2 bug22818_1 and bug22818_2 was corrected and tested in CR22818 branch. Test cases are OK. Test cases bugs modalg_2 bug22818_1 and bug22818_2 was pushed in git branches CR23832 and CR22818. Branch CR23832 is TESTED.

Date Modified	Username	Field	Change
2013-03-15 18:14	oan	New Issue
2013-03-15 18:14	oan	Assigned To	=> jgv
2013-03-18 10:22	~~mkv~~	Status	new => assigned
2013-03-18 11:13	~~mkv~~	Relationship added	child of 0022818
2013-03-18 11:17	~~abv~~	Assigned To	jgv => ika
2013-03-19 17:05	ika	Note Added: 0023803
2013-03-19 17:05	ika	Assigned To	ika => oan
2013-03-19 17:05	ika	Status	assigned => feedback
2013-03-19 18:06	oan	Note Added: 0023805
2013-03-19 18:07	oan	Note Edited: 0023805
2013-03-19 18:07	oan	Assigned To	oan => ika
2013-03-19 18:07	oan	Status	feedback => assigned
2013-03-21 15:47	ika	Note Added: 0023833
2013-03-21 15:47	ika	Assigned To	ika => gka
2013-03-21 15:47	ika	Status	assigned => resolved
2013-03-21 15:50	~~gka~~	Status	resolved => reviewed
2013-03-21 15:51	~~gka~~	Note Added: 0023834
2013-03-21 18:59	~~mkv~~	Assigned To	gka => mkv
2013-03-22 20:52	~~mkv~~	Note Added: 0023876
2013-03-22 20:53	~~mkv~~	Test case number	=> bugs modalg_2(007) bug22818_1, bug22818_2
2013-03-22 20:53	~~mkv~~	Assigned To	mkv => bugmaster
2013-03-22 20:53	~~mkv~~	Status	reviewed => tested
2013-03-25 18:39	ika	Changeset attached	=> occt master 903634d0
2013-03-25 18:39	ika	Assigned To	bugmaster => ika
2013-03-25 18:39	ika	Status	tested => verified
2013-03-25 18:39	ika	Resolution	open => fixed
2013-04-01 17:41	bugmaster	Target Version	=> 6.6.0
2013-04-23 13:35	~~aiv~~	Status	verified => closed
2013-04-29 15:24	~~aiv~~	Fixed in Version	=> 6.6.0

occt: master 903634d0 2013-03-21 11:34:43 ika Details Diff	0023832: sprops and triarea commands return incorrect value of a mass for shape of revolution incorrect way of creation faces in test scripts has been fixed	Affected Issues 0023832
	mod - tests/bugs/modalg_2/bug22818_1	Diff File
	mod - tests/bugs/modalg_2/bug22818_2	Diff File