MantisBT - Community
View Issue Details
0024117Community[OCCT] OCCT:Documentationpublic2013-08-20 16:182014-11-11 13:03
Timo 
bugmaster 
normalfeature 
closedfixed 
[OCCT] 6.5.4 
[OCCT] 6.8.0[OCCT] 6.8.0 
0024117: Enhance documentation of BRepMesh_IncrementalMesh
The meshing parameters of BRepMesh (BRepMesh_IncrementalMesh) are not documented.
About deflection and deviation angle, you can find something in the Express Mesh User's Guide chapter 1.1.3. "Meshing parameters and result quality", but that's not public. It could be used as a base for BRepMesh_IncementalMesh code documentation of parameters D and Ang.
In the Modeling Algorithms User's Guide I found a mention of BRepMesh in chapter 10 but it also doesn't explain the meshing parameters.

It would be good to add also information about restrictions of the minimum value of deflection, e.g. it is restricted below by doubled value of shape tolerance (0023918). OAN mentioned still other nuances of restrictions in comment 25348 (http://tracker.dev.opencascade.org/view.php?id=23918#c25348 [^]).
By documenting the restrictions, further issue tracker entries about this matter could be prevented.
No tags attached.
related to 0023918closed oan Community The triangulation of a face cannot be further refined by setting lower deflection in BRepMesh_IncrementalMesh 
Issue History
2013-08-20 16:18TimoNew Issue
2013-08-20 16:18TimoAssigned To => Timo
2013-08-20 16:18TimoIssue generated from: 0023918
2013-08-20 16:18TimoRelationship addedrelated to 0023918
2013-08-20 16:35TimoAssigned ToTimo => oan
2013-08-23 18:59TimoStatusnew => assigned
2013-09-17 13:16TimoDescription Updatedbug_revision_view_page.php?rev_id=5855#r5855
2013-12-21 10:13abvTarget Version6.7.0 => 6.7.1
2014-04-04 18:27abvTarget Version6.7.1 => 6.8.0
2014-05-19 10:23abvAssigned Tooan => azn
2014-05-21 13:02aznNote Added: 0029433
2014-05-21 13:03aznAssigned Toazn => abv
2014-05-21 13:03aznStatusassigned => resolved
2014-05-21 13:07abvAssigned Toabv => oan
2014-06-16 12:43oanNote Added: 0029780
2014-06-16 12:43oanAssigned Tooan => azn
2014-06-16 12:43oanStatusresolved => assigned
2014-06-23 10:37aznAssigned Toazn => oan
2014-06-23 10:37aznNote Added: 0029855
2014-06-24 14:40TimoNote Added: 0029870
2014-07-07 17:42oanNote Added: 0029990
2014-07-07 17:42oanAssigned Tooan => pdn
2014-07-07 17:42oanStatusassigned => resolved
2014-09-12 17:20pdnStatusresolved => reviewed
2014-09-16 13:35bugmasterAssigned Topdn => bugmaster
2014-09-16 13:35bugmasterStatusreviewed => tested
2014-09-19 13:42bugmasterChangeset attached => occt master f6d08024
2014-09-19 13:42bugmasterStatustested => verified
2014-09-19 13:42bugmasterResolutionopen => fixed
2014-09-30 11:34gitNote Added: 0032386
2014-10-15 16:58TimoNote Added: 0033150
2014-10-15 16:58TimoStatusverified => feedback
2014-10-15 16:58TimoResolutionfixed => reopened
2014-10-15 17:00TimoRelationship addedrelated to 0025378
2014-10-16 22:35abvAssigned Tobugmaster => oan
2014-10-29 06:52abvAssigned Tooan => ysn
2014-10-31 19:10ysnNote Added: 0033986
2014-10-31 19:10ysnAssigned Toysn => bugmaster
2014-10-31 19:10ysnStatusfeedback => resolved
2014-11-06 11:03ysnNote Edited: 0033986bug_revision_view_page.php?bugnote_id=33986#r8550
2014-11-06 11:58bugmasterStatusresolved => reviewed
2014-11-06 11:58bugmasterNote Added: 0034087
2014-11-06 11:58bugmasterStatusreviewed => tested
2014-11-06 15:35bugmasterStatustested => verified
2014-11-06 15:35bugmasterResolutionreopened => fixed
2014-11-11 12:42aivFixed in Version => 6.8.0
2014-11-11 13:03aivStatusverified => closed

Notes
(0029433)
azn   
2014-05-21 13:02   
Change pushed to CR24117, please review.
(0029780)
oan   
2014-06-16 12:43   
Dear AZN,

I propose to check the following places:
* "The *HLRBRep_PolyAlgo* algorithm works with triangulation of shapes." - IMHO, there is no reason in referring to component that have different weather than supposed by chapter name;
* "At the first step all edges from face discretized" - "are discretized"?
* please scale down the images as far as they become inconvenient for reading when browser window is not maximized to the full screen. Please, use other images from modeling_algos\images to choose the appropriate size.
(0029855)
azn   
2014-06-23 10:37   
Fixed. Please check.
(0029870)
Timo   
2014-06-24 14:40   
There is still a typo in dox/user_guides/modeling_algos/modeling_algos.md:

const Standard_Real anAnglularDeflection = 0.5;
should be:
const Standard_Real anAngularDeflection = 0.5;
(0029990)
oan   
2014-07-07 17:42   
Corrected.

Dear PDN,

Please verify the topic.
(0032386)
git   
2014-09-30 11:34   
Branch CR24117 has been deleted by inv.

SHA-1: 859910b1a024b7d5052a387975adb234d7eb38f7
(0033150)
Timo   
2014-10-15 16:58   
The two images modeling_algos_image056.png and modeling_algos_image057.png which were added to modeling_algos.md are not visible in the PDF of the Modeling Users's guide in OCCT 6.8.0 beta (see appended screenshot "image missing.PNG").

Still some comments about the text:
- After the introductory sentence the following could be added to show the main application of the mesh: "This triangulation is used to visualize the shape in shaded mode."
- Before "At the first step" I would start a new paragraph.
- Before "Linear deflection limits distance between triangles and face interior." I would add a sentence like "As a second step, the faces are tessellated." to make it clearer that now it speaks about face tesselation in contrast to edge tessellation before.
- The word "a" should probably be deleted from "This gives a meshes according to metrics"
(0033986)
ysn   
2014-10-31 19:10   
(edited on: 2014-11-06 11:03)
Fixed in branch CR25343_1.

(0034087)
bugmaster   
2014-11-06 11:58   
Fix was integrated with fix for issue 25343