MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0023640Community[OCCT] OCCT:Documentationpublic2012-12-13 18:452014-12-09 11:03
ReporterTimo 
Assigned Toabv 
PrioritynormalSeverityfeature 
StatusresolvedResolutionopen 
PlatformOSOS Version
Product Version[OCCT] 6.5.4 
Target Version[OCCT] 7.1.0Fixed in Version 
Summary0023640: Documentation for local sewing with BRepBuilderAPI_Sewing is missing
DescriptionIn http://www.opencascade.org/org/forum/thread_24171/?forum=3 [^] it was asked how local sewing can be performed in OpenCascade.

The answer was:
First you should Load() a context shape
and after use Add() for sub-shapes which should be locally sewed.

The forum supervisor wrote:
Unfortunately this feature still is not documented.
Therefore I suggest you to register the corresponding issue in Mantis Bug Tracker.

It would be helpful if a use case for local sewing is explained and how it can be used to get better performance than with global sewing.

Documentation about local sewing should be added to the Modeling Algorithms User's Guide as well as to the Doxygen Code documentation.
Steps To ReproduceNot applicable.
TagsNo tags attached.
Test case number
Attached Files

- Relationships

-  Notes
(0030424)
Timo (developer)
2014-07-25 14:42

I found some information about Local Sewing on http://www.opencascade.org/occt/overview/compocc7/sewing/sewing10/ [^]

It might be useful to for the proposed documentation enhancement:

"In case when a shape still has some unsewed faces or edges after sewing, it is possible to use local sewing with a greater tolerance.

Local sewing is especially good for open shells.
Local sewing allows to sew a hole in one part of the shape and to keep a required hole smaller than the specified working tolerance for local sewing. Local sewing is much faster than sewing on the whole shape.

All pre-existing connections of the whole shape are kept after local sewing."


There are also some other pieces of information about sewing on the website, but some of it is not up-to-date, like mentioning BRepAlgo_Sewing on http://www.opencascade.org/occt/overview/compocc7/sewing/sewing2/ [^]
(0033825)
ysn (developer)
2014-10-29 14:31

Dear Timo, actually, there is a section about sewing in the Technical Overview http://dev.opencascade.org/doc/overview/html/technical_overview.html#OCCT_TOVW_SECTION_4_9 [^]

As you can see, it is exactly based on the site contents.

Do you suggest somehow enhancing this?
(0033834)
Timo (developer)
2014-10-29 15:21

I rather meant to describe local sewing also in the Modeling User's guide and API reference documentation. I mean, how to actually apply local sewing using the Load- and Add-method. Not much text would be needed but at least to mention this possibility and the benefits.

For me, it wasn't clear in case of local sewing:
Load(): Can the context shape also be a compound containing a shell and additional faces to be sewed or does it have to be a shell containing all faces?
Add(): Should I add here the faces to be sewed or specify certain edges of faces which should be sewed?

Additionally, some parts seem not up-to-date in the Technical overview, at least BRepAlgo_Sewing is mentionend, but it doesn't exist anymore.
(0033837)
ysn (developer)
2014-10-29 15:39

OK. The section about sewing will be soon moved to the Modeling User's guide.

Dear Galina, could you do the following:

1) Review the description of Sewing to check if there is no outdated information.
2) Add a code example illustrating the usage of Local Sewing with proper comments. (And expand the section further if necessary).
(0034468)
Timo (developer)
2014-11-18 10:58

Here is an example code for local sewing which I received in the frame of a support query and which could be used in the documentation:

"In order to use local sewing it is necessary to create a compound from two full sewed shapes then load full compound using method BRepBuilderAPI_Sewing::Load().
After that it is necessary to add local sub-shapes which should be sewed using method BRepBuilderAPI_Sewing::Add(). The result of sewing can be got using method BRepBuilderAPI_Sewing::SewedShape().
For example:
//initial sewed shapes
TopoDS_Shape aS1, aS2;
TopoDS_Shape aComp;
BRep_Builder aB;
aB.MakeCompound(aComp);
aB.Add(aComp, aS1);
aB.Add(aComp, aS2);
................................
aSewing.Load(aComp);

//sub shapes which should be locally sewed
aSewing.Add(aF1);
aSewing.Add(aF2);
//performing sewing
aSewing.Perform();
//result shape
TopoDS_Shape aRes = aSewing.SewedShape();
(0035142)
git (administrator)
2014-12-08 16:22

Branch CR23640 has been created by ysn.

SHA-1: 3abaadd12ebffcff81a78133c27cc806bede2ea2


Detailed log of new commits:

Author: ysn
Date: Mon Dec 8 16:21:48 2014 +0300

    Fix for bug 23640. What's more important: merging the rest of Technical Overview content into User's Guides.
    However, Technical Overview itself is left intact and will be replaced by a shortened document only at the next stage.
(0035143)
ysn (developer)
2014-12-08 16:22

I've made a fix for bug 23640 in the corresponding branch, adding the requested example.

What's more important: merging the rest of Technical Overview content into User's Guides.
However, Technical Overview itself is left intact and will be replaced by a shortened document only at the next stage.
(0035144)
ysn (developer)
2014-12-08 16:30

Dear Michail, please, have a look at the proposed changes.
(0035175)
git (administrator)
2014-12-08 23:32

Branch CR23640 has been updated by msv.

SHA-1: a9b85ac3c0d3e57c080d4d0b40e3316ce015d576


Detailed log of new commits:

Author: msv
Date: Mon Dec 8 23:32:25 2014 +0300

    -Correct UG sections concerning Sewing.
    -Correct comments in the code referring to old location of sewing algorithm.

(0035176)
msv (developer)
2014-12-08 23:35

I have corrected the Sewing section of UG.
I have looked at other UGs only briefly.
(0035191)
ysn (developer)
2014-12-09 11:03

OK. Thank you for the input.

- Issue History
Date Modified Username Field Change
2012-12-13 18:45 Timo New Issue
2012-12-13 18:45 Timo Assigned To => atp
2012-12-13 19:13 szy Target Version => 6.6.0
2012-12-14 15:57 atp Assigned To atp => gka
2013-02-26 15:50 abv Target Version 6.6.0 => 6.7.0
2013-12-16 16:20 gka File Added: Surface_0.brep
2013-12-16 16:20 gka File Added: Surface_1.brep
2013-12-16 16:29 gka File Deleted: Surface_0.brep
2013-12-16 16:29 gka File Deleted: Surface_1.brep
2013-12-21 10:13 abv Target Version 6.7.0 => 6.7.1
2014-04-04 18:27 abv Target Version 6.7.1 => 6.8.0
2014-07-25 14:42 Timo Note Added: 0030424
2014-10-29 13:57 abv Assigned To gka => ysn
2014-10-29 13:57 abv Status new => assigned
2014-10-29 14:31 ysn Note Added: 0033825
2014-10-29 15:05 ysn Assigned To ysn => Timo
2014-10-29 15:05 ysn Status assigned => feedback
2014-10-29 15:21 Timo Note Added: 0033834
2014-10-29 15:22 Timo Assigned To Timo => ysn
2014-10-29 15:22 Timo Status feedback => assigned
2014-10-29 15:39 ysn Assigned To ysn => gka
2014-10-29 15:39 ysn Note Added: 0033837
2014-11-06 06:37 abv Target Version 6.8.0 => 7.1.0
2014-11-18 10:58 Timo Note Added: 0034468
2014-11-18 11:30 gka Assigned To gka => ysn
2014-12-08 16:22 git Note Added: 0035142
2014-12-08 16:22 ysn Note Added: 0035143
2014-12-08 16:29 ysn Assigned To ysn => msv
2014-12-08 16:30 ysn Note Added: 0035144
2014-12-08 16:30 ysn Status assigned => feedback
2014-12-08 23:32 git Note Added: 0035175
2014-12-08 23:35 msv Note Added: 0035176
2014-12-08 23:35 msv Assigned To msv => ysn
2014-12-08 23:35 msv Status feedback => resolved
2014-12-08 23:35 msv Steps to Reproduce Updated View Revisions
2014-12-09 11:02 ysn Assigned To ysn => abv
2014-12-09 11:03 ysn Note Added: 0035191


Copyright © 2000 - 2014 MantisBT Team
Powered by Mantis Bugtracker