MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Related Changesets ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0028508Open CASCADE[OCCT] OCCT:Modeling Algorithmspublic2017-03-06 13:062017-03-24 15:53
Reporteremv 
Assigned Tobugmaster 
PrioritynormalSeverityminor 
StatusverifiedResolutionfixed 
PlatformOSOS Version
Product Version[OCCT] 7.1.0 
Target Version[OCCT] 7.2.0*Fixed in Version 
Summary0028508: Make the CellsBuilder algorithm to work with multi-dimensional arguments
DescriptionCurrently, the Cells Builder algorithm supports only the arguments of the same type (only SOLIDs or only FACEs for example). The issue is to extend this algorithm to multi-dimensional mode.
Steps To Reproducebox b 10 10 10
plane p 5 5 5 1 0 0
mkface f p -10 10 -10 10
bclearobjects
bcleartools
baddobjects b f
bfillds
bcbuild rx
bcadd result b 1
checknbshapes result -solid 2

testgrid boolean cells_test
TagsNo tags attached.
Test case numberboolean cells_test J1, J2, J3, J4, J5, J6
Attached Files

- Relationships

-  Notes
(0064379)
git (administrator)
2017-03-16 14:06

Branch CR28508 has been created by emv.

SHA-1: 5e8e6a608fb222ed1693fd6d9fc2af2ad87de8ed


Detailed log of new commits:

Author: emv
Date: Fri Mar 10 15:13:48 2017 +0300

    0028508: Make the CellsBuilder algorithm to work with multi-dimensional arguments
    
    1. The CellsBuilder algorithm has been extended to work with multi-dimensional arguments.
    It has become possible not only simulate Boolean expressions, but also perform non-supported
    Boolean operations, like cutting face from solid, or fusing face with edge.
    
    2. Test cases with multi-dimensional input shapes have been created.
    
    3. Documentation has been updated.
(0064380)
emv (developer)
2017-03-16 14:07

Dear Mikhail, could you please review the git branch CR28508?
(0064537)
msv (developer)
2017-03-20 16:16

Remarks:

src\BOPAlgo\BOPAlgo_CellsBuilder.hxx
- 106: misprint "Waring"
- rename enum BOPAlgo_CellsBuilder_WarningStatusEnum into WarningStatus, and make it public.
- Replace word 'unification' with 'remove internal boundaries'
- make enum items more short, and revise their content and meaning.
- RemoveInternalBoundaries is the only method setting warning status. It should be reflected in description of the class.
- 145: add comment that this code is not needed if bUpdate is true.

src\BOPAlgo\BOPAlgo_CellsBuilder.cxx
- 791: extra words "should be created".
- accept output stream in DumpWarnings().

src\BOPTest\BOPTest_CellsCommands.cxx
- put warning statuses to "di".

dox\user_guides\boolean_operations\boolean_operations.md
- 2295: "impossible from Booleans point of view" => "impossible using standard Boolean fuse operation".
(0064538)
git (administrator)
2017-03-20 16:19

Branch CR28508 has been updated forcibly by emv.

SHA-1: 40fe0323d0565d41ccb2220ffe81a06e1f1f16bc
(0064539)
emv (developer)
2017-03-20 16:20

The git branch CR28508 has been rebased on current master.
(0064569)
git (administrator)
2017-03-21 10:44

Branch CR28508_1 has been created by emv.

SHA-1: 61a41fdf748f5457ff946b70098874673ba2e1bc


Detailed log of new commits:

Author: emv
Date: Fri Mar 10 15:13:48 2017 +0300

    0028508: Make the CellsBuilder algorithm to work with multi-dimensional arguments
    
    1. The CellsBuilder algorithm has been extended to work with multi-dimensional arguments.
    It has become possible not only simulate Boolean expressions, but also perform non-supported
    Boolean operations, like cutting face from solid, or fusing face with edge.
    
    2. Test cases with multi-dimensional input shapes have been created.
    
    3. Documentation has been updated.
(0064570)
emv (developer)
2017-03-21 10:46

The remarks have been considered. Please review the git branch CR28508_1.
(0064575)
msv (developer)
2017-03-21 11:45

Reviewed.
(0064620)
mkv (tester)
2017-03-22 15:47
edited on: 2017-03-22 15:50

Dear BugMaster,
Branch CR28508_1 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode.
SHA-1: 61a41fdf748f5457ff946b70098874673ba2e1bc

Number of compiler warnings:

occt component :
Linux: 0 (0 on master)
Windows: 0 (0 on master)
MacOS : 0 (0 on master)

products component :
Linux: 64 (64 on master)
Windows: 0 (0 on master)
MacOS : 1198

Regressions/Differences/Improvements:
No regressions/differences

Testing cases:
http://occt-tests/CR28508_1-master-OCCT/Debian70-64/boolean/cells_test/J1.html [^]
http://occt-tests/CR28508_1-master-OCCT/Windows-64-VC10/boolean/cells_test/J1.html [^]
boolean cells_test J1: OK
http://occt-tests/CR28508_1-master-OCCT/Debian70-64/boolean/cells_test/J2.html [^]
http://occt-tests/CR28508_1-master-OCCT/Windows-64-VC10/boolean/cells_test/J2.html [^]
boolean cells_test J2: OK
http://occt-tests/CR28508_1-master-OCCT/Debian70-64/boolean/cells_test/J3.html [^]
http://occt-tests/CR28508_1-master-OCCT/Windows-64-VC10/boolean/cells_test/J3.html [^]
boolean cells_test J3: OK
http://occt-tests/CR28508_1-master-OCCT/Debian70-64/boolean/cells_test/J4.html [^]
http://occt-tests/CR28508_1-master-OCCT/Windows-64-VC10/boolean/cells_test/J4.html [^]
boolean cells_test J4: OK
http://occt-tests/CR28508_1-master-OCCT/Debian70-64/boolean/cells_test/J5.html [^]
http://occt-tests/CR28508_1-master-OCCT/Windows-64-VC10/boolean/cells_test/J5.html [^]
boolean cells_test J5: OK
http://occt-tests/CR28508_1-master-OCCT/Debian70-64/boolean/cells_test/J6.html [^]
http://occt-tests/CR28508_1-master-OCCT/Windows-64-VC10/boolean/cells_test/J6.html [^]
boolean cells_test J6: OK

Testing on Linux:
occt component :
Total MEMORY difference: 92192314 / 92032328 [+0.17%]
Total CPU difference: 20148.03000000014 / 20187.16000000011 [-0.19%]
products component :
Total MEMORY difference: 31030037 / 31146794 [-0.37%]
Total CPU difference: 5468.999999999987 / 5412.149999999975 [+1.05%]
Testing on Windows:
occt component :
Total MEMORY difference: 57783854 / 57782265 [+0.00%]
Total CPU difference: 18883.53104749858 / 18782.551600198512 [+0.54%]
products component :
Total MEMORY difference: 22292937 / 22254740 [+0.17%]
Total CPU difference: 5417.587127899969 / 5421.409152399973 [-0.07%]

There are no differences in images found by testdiff.

(0064621)
mkv (tester)
2017-03-22 15:48

Dear BugMaster,
Branch CR28508_1 is TESTED.
(0064646)
git (administrator)
2017-03-23 14:08

Branch CR28508_2 has been created by emv.

SHA-1: 973bb448b17452705ed26acafa10034582ffee73


Detailed log of new commits:

Author: emv
Date: Fri Mar 10 15:13:48 2017 +0300

    0028508: Make the CellsBuilder algorithm to work with multi-dimensional arguments
    
    1. The CellsBuilder algorithm has been extended to work with multi-dimensional arguments.
    It has become possible not only simulate Boolean expressions, but also perform non-supported
    Boolean operations, like cutting face from solid, or fusing face with edge.
    
    2. Test cases with multi-dimensional input shapes have been created.
    
    3. Documentation has been updated.
(0064685)
git (administrator)
2017-03-24 13:47

Branch CR28508_2 has been updated by emv.

SHA-1: f122a90d439ce357581bb327ea20b22bf96eded3


Detailed log of new commits:

Author: emv
Date: Fri Mar 24 13:44:11 2017 +0300

    // Eliminating warnings.


- Related Changesets
occt: master e8b9db57
Timestamp: 2017-03-10 12:13:48
Author: emv
Committer: bugmaster
Details ] Diff ]
0028508: Make the CellsBuilder algorithm to work with multi-dimensional arguments

1. The CellsBuilder algorithm has been extended to work with multi-dimensional arguments.
It has become possible not only simulate Boolean expressions, but also perform non-supported
Boolean operations, like cutting face from solid, or fusing face with edge.

2. Test cases with multi-dimensional input shapes have been created.

3. Documentation has been updated.
add - dox/user_guides/boolean_operations/images/cells_algorithm_001.png Diff ] File ]
add - dox/user_guides/boolean_operations/images/cells_algorithm_006_2.png Diff ] File ]
add - dox/user_guides/boolean_operations/images/cells_algorithm_008.png Diff ] File ]
add - tests/boolean/cells_test/J6 Diff ] File ]
add - dox/user_guides/boolean_operations/images/cells_algorithm_002.png Diff ] File ]
mod - src/BOPAlgo/BOPAlgo_CellsBuilder.cxx Diff ] File ]
mod - src/BOPTest/BOPTest_CellsCommands.cxx Diff ] File ]
add - dox/user_guides/boolean_operations/images/cells_algorithm_007.png Diff ] File ]
mod - dox/user_guides/boolean_operations/boolean_operations.md Diff ] File ]
add - dox/user_guides/boolean_operations/images/cells_algorithm_005.png Diff ] File ]
add - tests/boolean/cells_test/J4 Diff ] File ]
add - tests/boolean/cells_test/J1 Diff ] File ]
add - dox/user_guides/boolean_operations/images/cells_algorithm_006_1.png Diff ] File ]
add - dox/user_guides/boolean_operations/images/cells_algorithm_003.png Diff ] File ]
add - dox/user_guides/boolean_operations/images/cells_algorithm_004.png Diff ] File ]
add - tests/boolean/cells_test/J3 Diff ] File ]
add - tests/boolean/cells_test/J5 Diff ] File ]
mod - src/BOPAlgo/BOPAlgo_CellsBuilder.hxx Diff ] File ]
add - tests/boolean/cells_test/J2 Diff ] File ]

- Issue History
Date Modified Username Field Change
2017-03-06 13:06 emv New Issue
2017-03-06 13:06 emv Assigned To => msv
2017-03-06 13:07 emv Assigned To msv => emv
2017-03-06 13:07 emv Status new => assigned
2017-03-06 13:07 emv Summary Making CellsBuilder algorithm multi-dimensional => Make the CellsBuilder algorithm to work with multi-dimensional arguments
2017-03-06 13:07 emv Description Updated View Revisions
2017-03-06 13:07 emv Description Updated View Revisions
2017-03-16 14:06 git Note Added: 0064379
2017-03-16 14:07 emv Note Added: 0064380
2017-03-16 14:07 emv Assigned To emv => msv
2017-03-16 14:07 emv Status assigned => resolved
2017-03-16 14:07 emv Steps to Reproduce Updated View Revisions
2017-03-20 16:16 msv Note Added: 0064537
2017-03-20 16:16 msv Assigned To msv => emv
2017-03-20 16:16 msv Status resolved => assigned
2017-03-20 16:19 git Note Added: 0064538
2017-03-20 16:20 emv Note Added: 0064539
2017-03-21 10:44 git Note Added: 0064569
2017-03-21 10:46 emv Note Added: 0064570
2017-03-21 10:46 emv Assigned To emv => msv
2017-03-21 10:46 emv Status assigned => resolved
2017-03-21 11:45 msv Note Added: 0064575
2017-03-21 11:45 msv Assigned To msv => bugmaster
2017-03-21 11:45 msv Status resolved => reviewed
2017-03-21 15:08 apv Assigned To bugmaster => mkv
2017-03-22 15:47 mkv Note Added: 0064620
2017-03-22 15:48 mkv Note Added: 0064621
2017-03-22 15:48 mkv Assigned To mkv => bugmaster
2017-03-22 15:48 mkv Status reviewed => tested
2017-03-22 15:48 mkv Test case number => boolean cells_test J1, J2, J3, J4, J5, J6
2017-03-22 15:50 mkv Note Edited: 0064620 View Revisions
2017-03-23 14:08 git Note Added: 0064646
2017-03-24 13:47 git Note Added: 0064685
2017-03-24 15:53 bugmaster Changeset attached => occt master e8b9db57
2017-03-24 15:53 bugmaster Status tested => verified
2017-03-24 15:53 bugmaster Resolution open => fixed


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker