MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0029334Open CASCADE[OCCT] OCCT:Modeling Algorithmspublic2017-11-19 18:012017-12-05 17:08
Reporterssv 
Assigned Toabv 
PrioritynormalSeverityminor 
StatusassignedResolutionopen 
PlatformWindowsOSVC++ 2013OS Version64 bit
Product Version 
Target Version[OCCT] 7.4.0*Fixed in Version 
Summary0029334: Simple offsets complicate geometry
DescriptionImagine you have a canonical shell like the one attached. Then you apply "simple offsets" to make a thick solid out of the shell. The problem is that the resulting geometry contains offset surfaces which are hardly reasonable for planes and cylinders. Essentially, it means that the resulting geometry should be simplified with canonical recognition which is a commercial product and not available for the community.

Probably this is as-designed behavior, but I have found it quite frustrating. We can make offsets more intelligent and do not build offset surfaces for canonic like cylinders and planes.
Steps To Reproduce// Make offset (use any offset value)
BRepOffsetAPI_MakeThickSolid mkOffset;
mkOffset.MakeThickSolidBySimple(inputShell, offsetValue);

// Get result
TopoDS_Shape result;
if ( mkOffset.IsDone() )
{
  result = mkOffset.Shape();
}
TagsNo tags attached.
Test case number
Attached Files? file icon inputShell.brep (175,179 bytes) 2017-11-19 18:01

- Relationships

-  Notes
(0072326)
git (administrator)
2017-11-20 08:51

Branch CR29334 has been created by abv.

SHA-1: ac8aa278ddc21df75a38e9ffb6f82ca17ac43e56


Detailed log of new commits:

Author: abv
Date: Mon Nov 20 08:51:32 2017 +0300

    0029334: Simple offsets complicate geometry
    
    Method BRepOffset::Surface() is used in simple offset algorithm to create offset surface, instead of direct and unconditional construction of new Geom_OffsetSurface instance.
(0072327)
abv (manager)
2017-11-20 08:52

Note that standard offset algorithm behaves as desired: offsetting planes and cylinders results in faces built on planes and cylinders, not offset surfaces. I see no reason why "simple offset" algorithm should not use corresponding method (BRepOffset::Surface()).
(0072338)
git (administrator)
2017-11-20 16:26

Branch CR29334 has been updated by abv.

SHA-1: cc28b067c88a9331169a7b4ac55b0a44dd3eac37


Detailed log of new commits:

Author: abv
Date: Mon Nov 20 16:26:05 2017 +0300

    Added test bugs modalg_7 bug29334

(0072339)
abv (manager)
2017-11-20 16:27

Fix pushed to CR29334, please review. Jenkins tests passed, see job CR29344-master
(0072347)
msv (developer)
2017-11-20 19:07

tests/bugs/modalg_7/bug29334
- It is not needed to call 'pload MODELING' in the test case. This plugin is loaded in bugs/begin.
(0072352)
aml (developer)
2017-11-21 07:40

No remarks from my side, please fix msv remark.

- Issue History
Date Modified Username Field Change
2017-11-19 18:01 ssv New Issue
2017-11-19 18:01 ssv Assigned To => msv
2017-11-19 18:01 ssv File Added: inputShell.brep
2017-11-20 08:51 git Note Added: 0072326
2017-11-20 08:52 abv Note Added: 0072327
2017-11-20 16:26 git Note Added: 0072338
2017-11-20 16:27 abv Note Added: 0072339
2017-11-20 16:27 abv Assigned To msv => aml
2017-11-20 16:27 abv Status new => resolved
2017-11-20 16:27 abv Target Version 7.4.0* => 7.3.0
2017-11-20 19:07 msv Note Added: 0072347
2017-11-21 07:40 aml Note Added: 0072352
2017-11-21 07:40 aml Assigned To aml => abv
2017-11-21 07:40 aml Status resolved => assigned
2017-12-05 17:08 msv Target Version 7.3.0 => 7.4.0*


Copyright © 2000 - 2018 MantisBT Team
Powered by Mantis Bugtracker