|Anonymous | Login||2018-12-16 11:56 MSK|
|My View | View Issues | Change Log | Roadmap|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0024258||Open CASCADE||[OCCT] OCCT:Modeling Algorithms||public||2013-10-14 21:00||2018-07-19 16:04|
|Product Version||[OCCT] 6.6.0|
|Target Version||[OCCT] 7.4.0*||Fixed in Version|
|Summary||0024258: Classes in TKBO toolkit lack documentation|
|Description||Significant part of public API (classes and their public methods) does not contain proper documentation. In particular:|
- Classes BOPAlgo_PaveFiller, BOPAlgo_Tools, BOPAlgo_WireEdgeSet, BOPAlgo_WireSplitter, BOPTools, BOPTools_EdgeSet, BOPTools_Set, BOPInt_Tools have no even superficial description of their purpose.
- In some classes not all public methods have description (like Images(), ShapesSD(), Splits(), Origins(), IsInterferred() in BOPAlgo_Builder).
|Tags||No tags attached.|
|Test case number||Not needed|
|I suppose, this issue should be assignet to emv.|
Branch CR24258 has been updated forcibly by aka.
Remarks to the first commit:
//! Prints the information about the shape =>
//! Prints the information about the input shapes
//! Sort pave blocks in common block according their tolerance =>
//! Sort pave blocks in common block so that the first one had the maximal tolerance
//! Returns true if pave block with index theI
//! in common block should be sorted
//! Finds pave block to be placed as first, and returns true
//! if it is to be sorted.
//! Initializes the pave block for vertices,which was
//! used during computing V/V interferences
//! Initializes pave blocks for edges connected with the given vertex
65: //! Add interfered pave blocks (<thePB1>, <thePB2>) to the map <theMLI>.
theMLI => theMILI
In declarations of methods FillMap change the prefix of variables "tne" to "the".
tag to distinct lines in comments where it is needed.
In 64, remove mentioning of partition.
At 92, remove Iterator declaration completely, as it is not implemented.
103: //! Sets auxiliary attributes to the container
What is "container"? It is better to say "algorithm".
120, 126: occurs => occur
126: two shapes has => two shapes have
127: one shape completely inside => one shape is located completely inside
117-128: it is needed to insert
Please pay attention firstly to the public methods. For this stage, you may not waste time to describe protected and private methods. It is for future commits, do not remove already done work.
|2013-10-14 21:00||kgv||New Issue|
|2013-10-14 21:00||kgv||Assigned To||=> ifv|
|2013-10-14 21:01||kgv||Relationship added||related to 0021762|
|2014-04-04 18:34||abv||Target Version||6.7.1 => 6.8.0|
|2014-10-17 03:26||abv||Target Version||6.8.0 => 7.1.0|
|2016-10-28 11:54||msv||Target Version||7.1.0 => 7.2.0|
|2016-11-23 11:17||ifv||Note Added: 0060710|
|2016-11-23 11:17||ifv||Assigned To||ifv => msv|
|2016-11-23 11:17||ifv||Status||new => feedback|
|2016-11-23 15:09||msv||Assigned To||msv => aka|
|2016-11-23 15:09||msv||Status||feedback => assigned|
|2016-12-13 16:10||git||Note Added: 0061621|
|2016-12-22 11:16||msv||Note Added: 0061988|
|2017-07-24 09:22||msv||Target Version||7.2.0 => 7.3.0|
|2017-10-18 19:11||mkv||Test case number||=> Not needed|
|2017-12-05 17:08||msv||Target Version||7.3.0 => 7.4.0*|
|2018-07-19 16:04||msv||Assigned To||aka => emv|
|Copyright © 2000 - 2018 MantisBT Team|