MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Related Changesets ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0029334Open CASCADE[OCCT] OCCT:Modeling Algorithmspublic2017-11-19 18:012019-09-07 16:27
Reporterssv 
Assigned Toabv 
PrioritynormalSeverityminor 
StatusverifiedResolutionfixed 
PlatformWindowsOSVC++ 2013OS Version64 bit
Product Version 
Target Version[OCCT] 7.4.0Fixed in Version 
Summary0029334: Modeling Algorithms - 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 numberbugs modalg_7 bug29334
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.
(0086606)
git (administrator)
2019-09-02 08:11

Branch CR29334_1 has been created by abv.

SHA-1: 090ef040125d33234fa6110f313987a63b4b8701


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.
    
    Added test bugs modalg_7 bug29334
(0086614)
abv (manager)
2019-09-02 14:56

Patch corrected according to remarks is rebased on master and tested, see branch CR29334_1 and Jenkins job CR29334-abv; please review
(0086621)
msv (developer)
2019-09-02 17:11

Reviewed.
(0086622)
apn (administrator)
2019-09-02 17:16

Combination -
OCCT branch : CR29334_1
master SHA - 090ef040125d33234fa6110f313987a63b4b8701
5f5b1aed1c6e139bbd34314eca77ae7abcd8895c
Products branch : master SHA - 34e9bf11c6b525830b5dee35839707da81e7d1ab
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.

Number of compiler warnings:
No new/fixed warnings

Regressions/Differences/Improvements:
No regressions/differences

CPU differences:
Debian80-64:
OCCT
Total CPU difference: 16169.9099999999 / 16167.270000000033 [+0.02%]
Products
Total CPU difference: 10504.550000000041 / 10484.930000000048 [+0.19%]
Windows-64-VC14:
OCCT
Total CPU difference: 18190.453125 / 18256.515625 [-0.36%]
Products
Total CPU difference: 11674.71875 / 12168.265625 [-4.06%]

Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
(0086943)
git (administrator)
2019-09-07 16:27

Branch CR29334_1 has been deleted by inv.

SHA-1: 090ef040125d33234fa6110f313987a63b4b8701
(0086959)
git (administrator)
2019-09-07 16:27

Branch CR29334 has been deleted by inv.

SHA-1: cc28b067c88a9331169a7b4ac55b0a44dd3eac37

- Related Changesets
occt: master 090ef040
Timestamp: 2017-11-20 05:51:32
Author: abv
Committer: abv
Details ] Diff ]
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.

Added test bugs modalg_7 bug29334
mod - src/BRepOffset/BRepOffset_SimpleOffset.cxx Diff ] File ]
add - tests/bugs/modalg_7/bug29334 Diff ] File ]

- 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
2019-08-12 16:37 msv Target Version 7.4.0 => 7.5.0*
2019-09-02 08:11 git Note Added: 0086606
2019-09-02 14:56 abv Note Added: 0086614
2019-09-02 14:56 abv Assigned To abv => msv
2019-09-02 14:56 abv Status assigned => resolved
2019-09-02 17:11 msv Note Added: 0086621
2019-09-02 17:11 msv Assigned To msv => bugmaster
2019-09-02 17:11 msv Status resolved => reviewed
2019-09-02 17:11 msv Target Version 7.5.0* => 7.4.0
2019-09-02 17:14 apn Test case number => bugs modalg_7 bug29334
2019-09-02 17:16 apn Note Added: 0086622
2019-09-02 17:16 apn Status reviewed => tested
2019-09-04 12:06 kgv Summary Simple offsets complicate geometry => Modeling Algorithms - Simple offsets complicate geometry
2019-09-07 11:38 abv Changeset attached => occt master 090ef040
2019-09-07 11:38 abv Assigned To bugmaster => abv
2019-09-07 11:38 abv Status tested => verified
2019-09-07 11:38 abv Resolution open => fixed
2019-09-07 16:27 git Note Added: 0086943
2019-09-07 16:27 git Note Added: 0086959


Copyright © 2000 - 2019 MantisBT Team
Powered by Mantis Bugtracker