MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Related Changesets ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0025209Open CASCADE[OCCT] OCCT:DRAWpublic2014-09-02 11:232017-04-21 14:59
Reporteroan 
Assigned Tobugmaster 
PrioritynormalSeverityfeature 
StatusverifiedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version[OCCT] 7.2.0*Fixed in Version 
Summary0025209: Draw command "normals" should be extended to show variable number of normals
DescriptionCurrent implementation of command displays a single normal taken at midpoint of a face. This behavior does not provide details necessary for investigation of problems related to face geometry.

Necessity in extended version of the command has came from bug 0023857. Here, the surface produced from points cloud is incorrect due to different directions of normals at surface boundaries comparing to normals taken within surface interior.

Screenshot Single_Normal.png shows behavior of command using old implementation;
Normals_At_MidVParam.png - new functionality disclosing a problem of surface geometry.
Additional information
and documentation updates
Command "normals" has been extended to show variable number of normals. User is able to specify number of normals along U and V axes in parametric space of face's surface as well as length of displayed normals. Using option -UseMesh it is also possible to show normals at each node of mesh associated to a shape.

Additionally, command "vnormals" has also been implemented to display normals in 3D view. Syntax of "vnormals" is similar to "normals" command.
TagsNo tags attached.
Test case number
Attached Filespng file icon Single_Normal.png (6,029 bytes) 2014-09-02 11:23
png file icon Normals_At_MidVParam.png (5,751 bytes) 2014-09-02 11:24
png file icon Surface_Normals.png (9,325 bytes) 2014-09-02 11:24

- Relationships
child of 0023857closedabv Community Degenerate surface normals occur on face boundary. 

-  Notes
(0031261)
git (administrator)
2014-09-02 11:28

Branch CR25209 has been created by oan.

SHA-1: b946f32fff6c9e5ea1727d0288062422cf825d7b


Detailed log of new commits:

Author: oan
Date: Tue Sep 2 11:28:06 2014 +0400

    0025209: Draw command "normals" should be extended to show variable number of normals
(0031262)
oan (developer)
2014-09-02 11:29

Branch CR25209 is ready to be reviewed.
(0031312)
git (administrator)
2014-09-03 15:47

Branch CR25209 has been updated by oan.

SHA-1: f141afddd48ad491c7379945696273efc72c2f37


Detailed log of new commits:

Author: oan
Date: Wed Sep 3 15:46:56 2014 +0400

    New command vnormals displaying geometric/mesh normals in 3D view has been implemented.

(0032378)
git (administrator)
2014-09-30 09:55

Branch CR25209_1 has been created by kgv.

SHA-1: 28eb6f7da8a012f28001ce08119d1a5cc34422e3


Detailed log of new commits:

Author: oan
Date: Tue Sep 30 09:55:27 2014 +0400

    0025209: Draw Harness - add command 'vnormals' and extend command 'normals' to show variable number of normals
(0056321)
git (administrator)
2016-07-31 13:10

Branch CR25209_1 has been updated forcibly by kgv.

SHA-1: 342604d22493761a43c0fb20be8bbc20066412fb
(0065350)
git (administrator)
2017-04-19 16:16

Branch CR25209_1 has been updated forcibly by kgv.

SHA-1: 0f4275226681bb3d6db188278f1d664e50834153
(0065351)
kgv (developer)
2017-04-19 16:16

Please test the patch.
(0065391)
mkv (tester)
2017-04-20 14:23

Dear BugMaster,
Branch CR25209_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: 0f4275226681bb3d6db188278f1d664e50834153

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 : 1178

Regressions/Differences/Improvements:
No regressions/differences

Testing cases:
Not needed

Testing on Linux:
occt component :
Total MEMORY difference: 91423722 / 90720015 [+0.78%]
Total CPU difference: 19885.13000000032 / 19728.57000000033 [+0.79%]
products component :
Total MEMORY difference: 30295065 / 30341918 [-0.15%]
Total CPU difference: 5391.659999999989 / 5356.299999999965 [+0.66%]
Testing on Windows:
occt component :
Total MEMORY difference: 57809474 / 57804189 [+0.01%]
Total CPU difference: 18224.005619798714 / 18136.473458698543 [+0.48%]
products component :
Total MEMORY difference: 22304556 / 22265369 [+0.18%]
Total CPU difference: 5313.394059999973 / 5359.788757399982 [-0.87%]

There are no differences in images found by testdiff.
(0065392)
mkv (tester)
2017-04-20 14:24

Dear BugMaster,
Branch CR25209_1 is TESTED.

- Related Changesets
occt: master 68ef63f9
Timestamp: 2016-07-31 10:09:37
Author: oan
Committer: bugmaster
Details ] Diff ]
0025209: Draw Harness - add command 'vnormals' and extend command 'normals' to show variable number of normals
mod - src/DBRep/DBRep_DrawableShape.hxx Diff ] File ]
mod - src/ViewerTest/ViewerTest_ObjectCommands.cxx Diff ] File ]
mod - src/DBRep/DBRep_DrawableShape.cxx Diff ] File ]
mod - src/DBRep/DBRep.cxx Diff ] File ]

- Issue History
Date Modified Username Field Change
2014-09-02 11:23 oan New Issue
2014-09-02 11:23 oan Assigned To => mkv
2014-09-02 11:23 oan File Added: Single_Normal.png
2014-09-02 11:24 oan File Added: Normals_At_MidVParam.png
2014-09-02 11:24 oan File Added: Surface_Normals.png
2014-09-02 11:24 oan Relationship added child of 0023857
2014-09-02 11:26 oan Assigned To mkv => oan
2014-09-02 11:28 git Note Added: 0031261
2014-09-02 11:28 oan Status new => assigned
2014-09-02 11:29 oan Note Added: 0031262
2014-09-02 11:29 oan Assigned To oan => kgv
2014-09-02 11:29 oan Status assigned => resolved
2014-09-03 15:47 git Note Added: 0031312
2014-09-04 19:04 oan Additional Information Updated View Revisions
2014-09-11 17:47 abv Target Version => 6.8.0
2014-09-26 10:54 san Relationship added related to 0025276
2014-09-30 09:55 git Note Added: 0032378
2014-10-17 08:56 kgv Target Version 6.8.0 => 7.1.0
2016-07-31 13:10 git Note Added: 0056321
2016-11-22 11:07 kgv Target Version 7.1.0 => 7.2.0*
2017-04-19 16:16 git Note Added: 0065350
2017-04-19 16:16 kgv Note Added: 0065351
2017-04-19 16:16 kgv Assigned To kgv => bugmaster
2017-04-19 16:16 kgv Status resolved => reviewed
2017-04-19 16:17 mkv Assigned To bugmaster => mkv
2017-04-20 14:23 mkv Note Added: 0065391
2017-04-20 14:24 mkv Note Added: 0065392
2017-04-20 14:24 mkv Assigned To mkv => bugmaster
2017-04-20 14:24 mkv Status reviewed => tested
2017-04-21 14:59 bugmaster Changeset attached => occt master 68ef63f9
2017-04-21 14:59 bugmaster Status tested => verified
2017-04-21 14:59 bugmaster Resolution open => fixed


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker