DiscussionsIssue archiveOCCT:Modeling Algorithms

Archived issue #0023981

Wrong section curves

Open CASCADEOCCT:Modeling Algorithmsclosed26 public notes

Search issues

Description

Intersection algorithm produces wrong intersection curves for the attached surfaces.

Steps to reproduce

restore s1 s1
restore s2 s2
intersect i s1 s2
xdistcs i_1 s1 0 1 100
xdistcs i_2 s1 0 1 100

Public activity

26 archived notes

Participants are labeled by their role within this record.

01Commenter 1
CR23981 is ready for reviewing
02Commenter 2
Dear bugmuster please test, because correction is minimal (one lene of code)
03Commenter 3
Dear Commenter 1,

Branch CR23981 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 2 (2 on master)
Windows: 11 (11 on master)

products component :
Linux: 0 (0 on master)
Windows: 63 (63 on master)

Regressions:
http://occt-tests/CR23981-master-occt/Debian60-64/summary.html
http://occt-tests/CR23981-master-occt/Windows-32-VC9/summary.html
bugs modalg_1(006) buc60899_1
draft angle(001) F8, G2, G8, H1, H2, H3, H4, H5, H6, H7, H8, H9, I3, I6, I9, J1, J2, J3, J5, K6, K7, K8

Improvements:
No improvements

Testing cases:
bugs moddata_3(013) bug23981

Testing on Linux:
Total MEMORY difference: 365835280 / 365637240
Total CPU difference: 44171.23000000091 / 44970.960000001185

Testing on Windows:
Total MEMORY difference: 418844468 / 416694304
Total CPU difference: 35840.609375 / 35693.875

There are not differences in images found by testdiff.
04Commenter 2
Dear mkv,

It's necessary to modify extension of shapes in test case (bug23981_s1.draw and bug23981_s2.draw). Shapes were modified and pushed to folder geom in public repository.
05Commenter 5
CR23981 is added by fix for regressions and is ready for reviewing
06Commenter 6
CR23981 is added by fix for regressions and is ready for testing.

Case bugs modalg_1(006) buc60899_1 is really inprovement, because result is more correct then for muster.
07Commenter 7
Dear Commenter 1,

Branch CR23981 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 2 (2 on master)
Windows: 7 (7 on master)

products component :
Linux: 0 (0 on master)
Windows: 63 (63 on master)

Regressions:
http://occt-tests/CR23981-master-occt/Debian60-64/summary.html
http://occt-tests/CR23981-master-occt/Windows-32-VC9/summary.html
bugs modalg_1(006) buc60899_1
draft angle(001) J5

Improvements:
No improvements

Testing cases:
bugs moddata_3(013) bug23981

Testing on Linux:
Total MEMORY difference: 366371084 / 366171512
Total CPU difference: 47402.27000000082 / 43367.39000000101

Testing on Windows:
Total MEMORY difference: 420969468 / 422797128
Total CPU difference: 41795.53125 / 36582.390625

There are not differences in images found by testdiff.
08Commenter 8
CR23981 is added by fix for regressions and is ready for rev. and test.
09Commenter 9
CR23981 is added by fix for regressions and is ready for testing
10Commenter 10
Dear Commenter 1,

Branch CR23981 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 920 (923 on master)
Windows: 851 (852 on master)

products component :
Linux: 188 (188 on master)
Windows: 287 (287 on master)

Regressions:
No regressions

Improvements:
No improvements

Testing cases:
http://occt-tests/CR23981-master-occt/Debian60-64/bugs/moddata_3/bug23981.html
http://occt-tests/CR23981-master-occt/Windows-32-VC9/bugs/moddata_3/bug23981.html
bugs moddata_3(013) bug23981 - BAD.

Testing on Linux:
Total MEMORY difference: 366251232 / 366043872
Total CPU difference: 44004.71000000052 / 42535.580000000824

Testing on Windows:
Total MEMORY difference: 426226252 / 428140588
Total CPU difference: 37486.296875 / 32972.90625

There are not differences in images found by testdiff.
11Commenter 11
To avoid missing files, new total branch CR23981_1 is created and pushed for testing.
12Commenter 12
To avoid missing files, new total branch CR23981_1 is created and pushed for testing.
13Commenter 13
Dear Commenter 1,

Branch CR23981_1 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 923 (923 on master)
Windows: 852 (852 on master)

products component :
Linux: 188 (188 on master)
Windows: 287 (287 on master)

Regressions:
http://occt-tests/CR23981-1-master-occt/Windows-32-VC9/bugs/modalg_1/buc60899_1.html
http://occt-tests/CR23981-1-master-occt/Debian60-64/bugs/modalg_1/buc60899_1.html
bugs modalg_1(006) buc60899_1

http://occt-tests/CR23981-1-master-occt/Debian60-64/draft/angle/E5.html
draft angle(001) E5

http://occt-tests/CR23981-1-master-occt/Windows-32-VC9/draft/angle/M3.html
http://occt-tests/CR23981-1-master-occt/Debian60-64/draft/angle/M3.html
draft angle(001) M3

http://jenkins-test-02.nnov.opencascade.com:8080/user/mnt/my-views/view/CR23981_1/job/mnt-CR23981_1-master_products_tests_windows_start/label=windows_test,tests_group=parasolid,tests_subgroup=001/2/HTML_Report/?
http://jenkins-test-02.nnov.opencascade.com:8080/user/mnt/my-views/view/CR23981_1/job/mnt-CR23981_1-master_products_tests_linux_start/label=Debian60-64,tests_group=parasolid,tests_subgroup=001/3/HTML_Report/?
parasolid doc_1(001) A1, A3, A4

http://jenkins-test-02.nnov.opencascade.com:8080/user/mnt/my-views/view/CR23981_1/job/mnt-CR23981_1-master_products_tests_windows_start/label=windows_test,tests_group=parasolid,tests_subgroup=002/2/HTML_Report/?
http://jenkins-test-02.nnov.opencascade.com:8080/user/mnt/my-views/view/CR23981_1/job/mnt-CR23981_1-master_products_tests_linux_start/label=Debian60-64,tests_group=parasolid,tests_subgroup=002/3/HTML_Report/?
parasolid doc_2(002) A1, A3

http://jenkins-test-02.nnov.opencascade.com:8080/user/mnt/my-views/view/CR23981_1/job/mnt-CR23981_1-master_products_tests_windows_start/label=windows_test,tests_group=parasolid,tests_subgroup=003/2/HTML_Report/?
http://jenkins-test-02.nnov.opencascade.com:8080/user/mnt/my-views/view/CR23981_1/job/mnt-CR23981_1-master_products_tests_linux_start/label=Debian60-64,tests_group=parasolid,tests_subgroup=003/3/HTML_Report/?
parasolid doc_3(003) A4, B1, B7, D9

Improvements:
No improvements

Testing cases:
http://occt-tests/CR23981-1-master-occt/Windows-32-VC9/bugs/moddata_3/bug23981.html
http://occt-tests/CR23981-1-master-occt/Debian60-64/bugs/moddata_3/bug23981.html
bugs moddata_3(013) bug23981 - BAD.

Testing on Linux:
Total MEMORY difference: 366180980 / 366026128
Total CPU difference: 43314.400000000074 / 42534.36000000082

Testing on Windows:
Total MEMORY difference: 426133212 / 428037772
Total CPU difference: 41815.078125 / 32971.15625

There are not differences in images found by testdiff.

14Commenter 14
CR23981_1 was added some modifications and ready for testing and reviewing

15Commenter 15
CR23981_1 was added some modifications and ready for testing and reviewing

Dear MKV bugs modalg_1(006) buc60899_1
is really improvement!!!
Change please length of section in test condition.

16Commenter 16
CR23981_2 was added some modifications and ready for testing and reviewing

Dear MKV bugs modalg_1(006) buc60899_1
is really improvement!!!
Change please length of section in test condition.

17Commenter 17
CR23981_2 was created and ready for testing

Dear MKV bugs modalg_1(006) buc60899_1
is really improvement!!!
Change please length of section in test condition.

18Commenter 18
Dear Commenter 1,

Branch CR23981_2 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 783 (781 on master)
Windows: 387 (386 on master)

products component :
Linux: 189 (191 on master)
Windows: 287 (287 on master)

Regressions:
No regressions in occt part of test cases.

Improvements:
http://occt-tests/CR23981-2-master-occt/Debian60-64/bugs/modalg_1/buc60899_1.html
http://occt-tests/CR23981-2-master-occt/Windows-32-VC9/bugs/modalg_1/buc60899_1.html
bugs modalg_1(006) buc60899_1

Testing cases:
http://occt-tests/CR23981-2-master-occt/Debian60-64/bugs/moddata_3/bug23981.html
http://occt-tests/CR23981-2-master-occt/Windows-32-VC9/bugs/moddata_3/bug23981.html
bugs moddata_3(013) bug23981 - OK.

Testing on Linux:
Total MEMORY difference: 365513752 / 365355036
Total CPU difference: 49055.760000000446 / 54103.25000000012

Testing on Windows:
Total MEMORY difference: 429005908 / 429543216
Total CPU difference: 31234.46875 / 36543.4375

There is following improvement in images found by testdiff.
http://occt-tests/CR23981-2-master-occt/Debian60-64/diff-Debian60-64.html
IMAGE bugs modalg_1 buc60899_1: buc60899_1.png differs

19Commenter 19
20Commenter 20
test results of xt-translator looks OK, except 'parasolid doc_1 A4' on linux.
I tested this test case on Mandriva2010, but don't get same results.
mkv, could you please retest it?
21Commenter 21
Dear isn,
could you please rebase branch CR23981_2 to current master, there are conflict files.
22Commenter 22
CR23981_4 is ready for testing
it is new branch that got by rebase CR23981_2 on master
23Commenter 23
CR23981_4 is ready for testing
it is new branch that got by rebase CR23981_2 on master
24Commenter 24
Dear Commenter 1,

Branch CR23981_4 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 425 (426 on master)
Windows: 24 (24 on master)

products component :
Linux: 1189 (190 on master)
Windows: 287 (287 on master)

Regressions/Differences:
http://occt-tests/CR23981-4-master-occt/Debian60-64/bugs/modalg_1/buc60899_1.html
http://occt-tests/CR23981-4-master-occt/Windows-32-VC9/bugs/modalg_1/buc60899_1.html
bugs modalg_1(006) buc60899_1: FAILED

Testing cases:
http://occt-tests/CR23981-4-master-occt/Debian60-64/bugs/moddata_3/bug23981.html
http://occt-tests/CR23981-4-master-occt/Windows-32-VC9/bugs/moddata_3/bug23981.html
bugs moddata_3(013) bug23981: OK

Testing on Linux:
Total MEMORY difference: 365407508 / 366128040
Total CPU difference: 44005.18000000126 / 44692.3600000013

Testing on Windows:
Total MEMORY difference: 433833216 / 433414388
Total CPU difference: 32352.125 / 34607.15625

There are not differences in images found by testdiff.
25Commenter 25
Dear MKV, I have tried to write (see notes 24895, 25394, 25401) that bugs modalg_1(006) buc60899_1: FAILED
is IMPROVEMENT.
See pictures on muster for comparing - only part of section is made.
Now we have complete section.
26Commenter 26
Dear Commenter 1,

bugs modalg_1(006) buc60899_1 was corrected and pushed to git repository.

Issue CR23981 is TESTED.

Related records