MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Related Changesets ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0032091Open CASCADE[OCCT] OCCT:Visualizationpublic2021-01-28 10:562021-04-25 12:24
Reporternds 
Assigned Tobugmaster 
PrioritynormalSeverityfeature 
StatusverifiedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version[OCCT] 7.6.0*Fixed in Version 
Summary0032091: Visualization - possibility to redefine Graphic3d_TransformPers
DescriptionAllow redefining Apply/Compute of Graphic3d_TransformPers on a custom application level.

Here, we need to brainstorm about way to redefine template method Apply which is used both for Standard_Real and Standard_ShortReal now.
TagsNo tags attached.
Test case numberv3d/bugs/bug32091
Attached Filespng file icon snap_1_fit.png (3,054 bytes) 2021-04-19 16:05
png file icon snap_2_zoomed_8.png (3,052 bytes) 2021-04-19 16:06
png file icon snap_3_zoomed_0.2.png (2,888 bytes) 2021-04-19 16:07

- Relationships
parent of 0032135newnds Visualization - possibility to redefine zoom persistent scale compute in Graphic3d_TransformPers 
Not all the children of this issue are yet resolved or closed.

-  Notes
(0098468)
git (administrator)
2021-01-28 10:56

Branch CR32091 has been created by nds.

SHA-1: 0a70041e18aa8a8a728bbf95a3884dfd35de7223


Detailed log of new commits:

Author: nds
Date: Thu Jan 28 10:57:54 2021 +0300

    0032091: Visualization - possibility to redefine Graphic3d_TransformPers
(0100456)
git (administrator)
2021-04-19 16:02

Branch CR32091_1 has been created by nds.

SHA-1: 277f1e160358a8d2279a5f382e181a326ad4f7aa


Detailed log of new commits:

Author: nds
Date: Thu Jan 28 10:57:54 2021 +0300

    0032091: Visualization - possibility to redefine Graphic3d_TransformPers
(0100457)
git (administrator)
2021-04-19 16:10

Branch CR32091_1 has been updated forcibly by nds.

SHA-1: c602efa39064cbb4fd997ce1565f697857ff4ca9
(0100458)
nds (developer)
2021-04-19 16:15

Dear Kirill,

please, review.

Best regards, Natasha
(0100460)
kgv (developer)
2021-04-19 16:53

+// own includes

Please drop redundant comments.

+//purpose  : .

Please remove redundant trailing spaces.

+  ~Graphic3d_TransformPersScaledAbove() {}

virtual (just for consistency).

+  Standard_EXPORT Standard_Boolean HasMinimumSize();

const.

+++ b/tests/v3d/bugs/bug32091

Please move test into tests/v3d/trsf grid.

+  if (aScale < myScale )

Unintended space.

+  Standard_Real aScale = Graphic3d_TransformPers::persistentScale (theCamera, theViewportWidth, theViewportHeight);


Tip: base_type:: could be used instead of Graphic3d_TransformPers::.
(0100463)
git (administrator)
2021-04-19 20:07

Branch CR32091_1 has been updated by nds.

SHA-1: 6e012f17c6f8b5b5e21651248c94af787e355b61


Detailed log of new commits:

Author: nds
Date: Mon Apr 19 20:07:31 2021 +0300

    fix remarks

(0100466)
git (administrator)
2021-04-19 21:51

Branch CR32091_2 has been created by nds.

SHA-1: 2b84d482698697002ae8e1c7177c59af82c313f9


Detailed log of new commits:

Author: nds
Date: Thu Jan 28 10:57:54 2021 +0300

    0032091: Visualization - possibility to redefine Graphic3d_TransformPers
(0100469)
nds (developer)
2021-04-20 09:44

Dear Kirill,
remarks are corrected, please review once again.
Jenkins job: http://jenkins-test-12.nnov.opencascade.com/view/CR32091-master-nds/ [^]

Thank you in advance,
Natasha
(0100509)
kgv (developer)
2021-04-22 11:03

Natalia,

please correct the patch by removing unexpected non-latin symbols:
  22 //! This persistenñe works only when the camera scale value is below the scale value of this persistence.



   http://jenkins-test-12.nnov.opencascade.com/view/WEEK-16/view/GIT/job/WEEK-16-OCCT-check-code/ [^] Error: 
File(s) with NON-ASCII symbols :

        Non-ASC chars detected: src/Graphic3d/Graphic3d_TransformPersScaledAbove.hxx : 22
(0100536)
bugmaster (administrator)
2021-04-23 18:41

Corrected
(0100551)
bugmaster (administrator)
2021-04-25 11:00

Combination -
OCCT branch : IR-2021-04-23
master SHA - fd2efe5428b3b2b9d300b0612c0a53ee2486ba64
a87b7ddc8cb44606b91e3f37113847c3f5f50fdc
Products branch : IR-2021-04-23 SHA - 740db2b9c0429fd95b1e8cf3790001245aaec02b
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: 17901.710000000523 / 17988.35000000045 [-0.48%]
Products
Total CPU difference: 11558.800000000136 / 11564.760000000093 [-0.05%]
Windows-64-VC14:
OCCT
Total CPU difference: 5522.40625 / 5520.953125 [+0.03%]
Products
Total CPU difference: 12871.203125 / 12932.890625 [-0.48%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
(0100559)
git (administrator)
2021-04-25 12:24

Branch CR32091_2 has been deleted by inv.

SHA-1: 2b84d482698697002ae8e1c7177c59af82c313f9
(0100560)
git (administrator)
2021-04-25 12:24

Branch CR32091_1 has been deleted by inv.

SHA-1: 6e012f17c6f8b5b5e21651248c94af787e355b61
(0100569)
git (administrator)
2021-04-25 12:24

Branch CR32091 has been deleted by inv.

SHA-1: 0a70041e18aa8a8a728bbf95a3884dfd35de7223

- Related Changesets
occt: master b06da8f1
Timestamp: 2021-01-28 07:57:54
Author: nds
Committer: bugmaster
Details ] Diff ]
0032091: Visualization - possibility to redefine Graphic3d_TransformPers
mod - src/AIS/AIS_Plane.cxx Diff ] File ]
mod - src/AIS/AIS_Plane.hxx Diff ] File ]
mod - src/Graphic3d/FILES Diff ] File ]
mod - src/Graphic3d/Graphic3d_TransformPers.hxx Diff ] File ]
add - src/Graphic3d/Graphic3d_TransformPersScaledAbove.cxx Diff ] File ]
add - src/Graphic3d/Graphic3d_TransformPersScaledAbove.hxx Diff ] File ]
mod - src/ViewerTest/ViewerTest_ObjectCommands.cxx Diff ] File ]
add - tests/v3d/trsf/bug32091 Diff ] File ]

- Issue History
Date Modified Username Field Change
2021-01-28 10:56 nds New Issue
2021-01-28 10:56 nds Assigned To => nds
2021-01-28 10:56 git Note Added: 0098468
2021-02-16 17:34 nds Severity minor => feature
2021-02-16 17:56 nds Relationship added parent of 0032135
2021-02-16 18:16 nds Description Updated View Revisions
2021-04-19 16:02 git Note Added: 0100456
2021-04-19 16:03 nds File Added: snap_1_fit.png
2021-04-19 16:03 nds File Added: snap_1_zoomed_8.png
2021-04-19 16:04 nds File Added: snap_1_zoomed_0.2.png
2021-04-19 16:04 nds File Deleted: snap_1_fit.png
2021-04-19 16:05 nds File Added: snap_1_fit.png
2021-04-19 16:06 nds File Deleted: snap_1_zoomed_0.2.png
2021-04-19 16:06 nds File Deleted: snap_1_zoomed_8.png
2021-04-19 16:06 nds File Added: snap_2_zoomed_8.png
2021-04-19 16:07 nds File Added: snap_3_zoomed_0.2.png
2021-04-19 16:10 git Note Added: 0100457
2021-04-19 16:15 nds Note Added: 0100458
2021-04-19 16:15 nds Assigned To nds => kgv
2021-04-19 16:15 nds Status new => resolved
2021-04-19 16:53 kgv Note Added: 0100460
2021-04-19 16:54 kgv Assigned To kgv => nds
2021-04-19 16:54 kgv Status resolved => assigned
2021-04-19 20:07 git Note Added: 0100463
2021-04-19 21:51 git Note Added: 0100466
2021-04-20 09:44 nds Note Added: 0100469
2021-04-20 09:44 nds Assigned To nds => kgv
2021-04-20 09:44 nds Status assigned => resolved
2021-04-20 09:58 kgv Assigned To kgv => bugmaster
2021-04-20 09:58 kgv Status resolved => reviewed
2021-04-22 11:03 kgv Note Added: 0100509
2021-04-22 11:03 kgv Assigned To bugmaster => nds
2021-04-22 11:03 kgv Status reviewed => feedback
2021-04-23 18:41 bugmaster Note Added: 0100536
2021-04-23 18:41 bugmaster Assigned To nds => bugmaster
2021-04-23 18:41 bugmaster Status feedback => reviewed
2021-04-25 11:00 bugmaster Note Added: 0100551
2021-04-25 11:00 bugmaster Status reviewed => tested
2021-04-25 11:14 bugmaster Test case number => v3d/bugs/bug32091
2021-04-25 11:55 bugmaster Changeset attached => occt master b06da8f1
2021-04-25 11:55 bugmaster Status tested => verified
2021-04-25 11:55 bugmaster Resolution open => fixed
2021-04-25 12:24 git Note Added: 0100559
2021-04-25 12:24 git Note Added: 0100560
2021-04-25 12:24 git Note Added: 0100569


Copyright © 2000 - 2021 MantisBT Team
Powered by Mantis Bugtracker