MantisBT - Community
View Issue Details
0027828Community[OCCT] OCCT:Documentationpublic2016-08-31 12:512016-12-09 16:39
Timo 
apn 
normalminor 
closedfixed 
[OCCT] 7.0.0 
[OCCT] 7.1.0[OCCT] 7.1.0 
Not needed
0027828: Reference documentation of AIS_Trihedron contains invalid information about default size.
The reference documentation of AIS_Trihedron says in several places that its default size is 100 mm. But this is not true.

You may have a look at the following functions:

Standard_Real AIS_Trihedron::Size() const
{
  return myDrawer->DatumAspect()->FirstAxisLength();
}


Prs3d_DatumAspect::Prs3d_DatumAspect()
: myDrawFirstAndSecondAxis (Standard_True),
  myDrawThirdAxis (Standard_True),
  myToDrawLabels (Standard_True),
  myFirstAxisLength (10.0),...

So, it dependends on its datum aspect, which defaults to 10.
NA
No tags attached.
Issue History
2016-08-31 12:51TimoNew Issue
2016-08-31 12:51TimoAssigned To => ysn
2016-11-08 15:39gitNote Added: 0060029
2016-11-08 15:42ysnNote Added: 0060030
2016-11-08 15:42ysnAssigned Toysn => Timo
2016-11-08 15:42ysnStatusnew => resolved
2016-11-08 15:42ysnSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=15238#r15238
2016-11-08 17:02TimoNote Added: 0060041
2016-11-08 17:02TimoAssigned ToTimo => ysn
2016-11-08 17:02TimoStatusresolved => assigned
2016-11-08 17:37gitNote Added: 0060049
2016-11-08 17:40ysnNote Added: 0060050
2016-11-08 17:40ysnAssigned Toysn => bugmaster
2016-11-08 17:40ysnStatusassigned => resolved
2016-11-16 11:22kgvNote Added: 0060410
2016-11-16 11:22kgvAssigned Tobugmaster => ysn
2016-11-16 11:22kgvStatusresolved => assigned
2016-11-16 11:29kgvNote Edited: 0060410bug_revision_view_page.php?bugnote_id=60410#r15338
2016-11-16 17:08gitNote Added: 0060438
2016-11-16 17:09ysnNote Added: 0060439
2016-11-16 17:09ysnAssigned Toysn => kgv
2016-11-16 17:09ysnStatusassigned => resolved
2016-11-16 17:11kgvNote Added: 0060440
2016-11-16 17:11kgvAssigned Tokgv => bugmaster
2016-11-16 17:11kgvStatusresolved => reviewed
2016-11-16 18:34apnTest case number => Not needed
2016-11-16 18:34apnStatusreviewed => tested
2016-11-18 16:37apnChangeset attached => occt master f3495b01
2016-11-18 16:37apnAssigned Tobugmaster => apn
2016-11-18 16:37apnStatustested => verified
2016-11-18 16:37apnResolutionopen => fixed
2016-12-07 11:30gitNote Added: 0061246
2016-12-07 11:30gitNote Added: 0061247
2016-12-09 16:30aivStatusverified => closed
2016-12-09 16:39aivFixed in Version => 7.1.0

Notes
(0060029)
git   
2016-11-08 15:39   
Branch CR27828 has been created by ysn.

SHA-1: a9da3b3f16e8ee14a98dbba9c8693639c5226af8


Detailed log of new commits:

Author: ysn
Date: Tue Nov 8 15:38:37 2016 +0300

    Fixed the description of AIS_Trihedron
(0060030)
ysn   
2016-11-08 15:42   
Please, review.
(0060041)
Timo   
2016-11-08 17:02   
The "100" mm still occur in AIS_Trihedron.hxx at the description of methods:
- HasOwnSize
- SetSize
(0060049)
git   
2016-11-08 17:37   
Branch CR27828 has been updated by ysn.

SHA-1: 044841381ee7b26070d343d4fbcd82304d81b9bb


Detailed log of new commits:

Author: ysn
Date: Tue Nov 8 17:37:34 2016 +0300

    Merge branch 'CR27828' of git.dev.opencascade.org:occt into CR27828

Author: ysn
Date: Tue Nov 8 15:38:37 2016 +0300

    0027828: Reference documentation of AIS_Trihedron contains invalid information about default size.
    
    Fixed the description of AIS_Trihedron

(0060050)
ysn   
2016-11-08 17:40   
Please, integrate.
(0060410)
kgv   
2016-11-16 11:22   
(edited on: 2016-11-16 11:29)
> Please, integrate.
Please prepare patch following bug advancement workflow (e.g. as single commit over current master branch and without any merges).
Bug in resolved state should be assigned to reviewer, not bugmaster.

(0060438)
git   
2016-11-16 17:08   
Branch CR27828_1 has been created by ysn.

SHA-1: 224e6015da6077b0ec01092beb17027c270f2978


Detailed log of new commits:

Author: ysn
Date: Wed Nov 16 17:08:24 2016 +0300

    0027828: Reference documentation of AIS_Trihedron contains invalid information about default size.
    
    Fixed the description of AIS_Trihedron
(0060439)
ysn   
2016-11-16 17:09   
Please, review.
(0060440)
kgv   
2016-11-16 17:11   
Please integrate the patch, regression testing is not needed.
(0061246)
git   
2016-12-07 11:30   
Branch CR27828 has been deleted by kgv.

SHA-1: 044841381ee7b26070d343d4fbcd82304d81b9bb
(0061247)
git   
2016-12-07 11:30   
Branch CR27828_1 has been deleted by kgv.

SHA-1: 224e6015da6077b0ec01092beb17027c270f2978