3 years ago0027961: Visualization - remove unused and no more working OpenGl_AVIWriter
kgv [Thu, 13 Oct 2016 13:28:34 +0000]
0027961: Visualization - remove unused and no more working OpenGl_AVIWriter

3 years ago0027604: Application Framework - memory is not released after closing XBF file
mpv [Thu, 20 Oct 2016 13:26:25 +0000]
0027604: Application Framework - memory is not released after closing XBF file

Virtual method *BinLDrivers_DocumentRetrievalDriver::Clear* now allows to Clear a cash data accumulated during a reading.
So, in successor class BinDrivers_DocumentRetrievalDriver this allows clearing the BinMNaming_NamedShapeDriver set of stored shapes when it is not needed anymore.

3 years ago0027963: Coding Rules - eliminate GCC compiler warnings -Wmissing-field-initializers...
kgv [Thu, 13 Oct 2016 18:09:51 +0000]
0027963: Coding Rules - eliminate GCC compiler warnings -Wmissing-field-initializers in OSD_File.cxx

3 years ago0027945: Visualization - handle correctly view clipping planes within zoom-persistent...
kgv [Thu, 13 Oct 2016 11:01:19 +0000]
0027945: Visualization - handle correctly view clipping planes within zoom-persistent objects

OpenGl_Structure::Render() and SelectMgr_ViewerSelector::checkOverlap()
now clip entire zoom/rotate persistence object
by checking anchor point with global clipping planes.

Image dump has been added to the new test case.

3 years ago0027953: Foundation Classes - eliminate global variables from OSD_Path
kgv [Wed, 12 Oct 2016 09:46:48 +0000]
0027953: Foundation Classes - eliminate global variables from OSD_Path

3 years ago0027931: Coding rules, Precision.hxx - eliminate dead code within PRECISION_OBSOLETE...
kgv [Wed, 5 Oct 2016 10:52:48 +0000]
0027931: Coding rules, Precision.hxx - eliminate dead code within PRECISION_OBSOLETE macros

3 years ago0027943: Visualization - fix broken shading by positional light for object with local...
kgv [Sun, 9 Oct 2016 17:48:16 +0000]
0027943: Visualization - fix broken shading by positional light for object with local transformation

OpenGl_ShaderManager, per-vertex shading and per-fragment shading
- do not apply model-world matrix on the light sources
which are expected to be defined in world-space.

3 years ago0027952: Visualization - do not reset RayTracing state on changing structures not...
kgv [Tue, 11 Oct 2016 19:55:25 +0000]
0027952: Visualization - do not reset RayTracing state on changing structures not in main Z-Layer

3 years ago0027824: Parasolid XMT file generates weird geometry after conversion
anv [Thu, 1 Sep 2016 09:29:14 +0000]
0027824: Parasolid XMT file generates weird geometry after conversion

Fixed adapter for C0 offsets

3 years ago0027930: XMT file conversion loops infinitely
nbv [Thu, 13 Oct 2016 10:48:38 +0000]
0027930: XMT file conversion loops infinitely

Now, the algorithm tries to estimate U- and V-ranges of future intersection curve(s) on the surface. This information is used in stop-criterium of the algorithm instead of full surface range used earlier. It allows reducing dependencies of intersection result on the surface ranges.
Tuning of test case bugs/modalg_6/bug27937_1

3 years ago0027946: Out of range item access in offset algorithm
aml [Mon, 10 Oct 2016 12:08:36 +0000]
0027946: Out of range item access in offset algorithm

Out of range access has been eliminated.

3 years ago0027619: Improvement on manuipulation of data by TPrsStd_AISPresentation attribute.
vro [Thu, 13 Oct 2016 10:45:03 +0000]
0027619: Improvement on manuipulation of data by TPrsStd_AISPresentation attribute.

The branch contains a fix on Undo/Redo for TPrsStd_AISPresentation attribute + an opportunity to deselect the interactive object.
getData() now calls FindAttribute(), the field myData disappeared.
A fix of regression on bug73_3.
A cleverer solution.

3 years ago0027941: Foundation Classes - NCollection_Vector::Iterator::offsetV() does not match...
kgv [Fri, 7 Oct 2016 15:49:34 +0000]
0027941: Foundation Classes - NCollection_Vector::Iterator::offsetV() does not match NCollection_Vector::end()

3 years ago0027944: Visualization, V3d_View - filter duplicates within ::AddClipPlane()
kgv [Sun, 9 Oct 2016 18:59:58 +0000]
0027944: Visualization, V3d_View - filter duplicates within ::AddClipPlane()

3 years ago0027915: Foundation Classes - remove the class NCollection_QuickSort
ski [Thu, 13 Oct 2016 10:40:12 +0000]
0027915: Foundation Classes - remove the class NCollection_QuickSort

Class NCollection_QuickSort was removed.
Direction of dimension lines was corrected.
Updated porting notes.
minor corrections in AIS

3 years ago0027895: Correction in the constructor Extrema_ExtElC::Extrema_ExtElC (const gp_Lin...
nbv [Thu, 22 Sep 2016 13:46:12 +0000]
0027895: Correction in the constructor Extrema_ExtElC::Extrema_ExtElC (const gp_Lin&,const gp_Lin&,const Standard_Real)

Computation algorithm is made more simple.
Adjusting some test cases according to their new behavior.

3 years ago0026894: Intersection algorithm between two infinite cylinders is hanging
nbv [Thu, 6 Oct 2016 09:57:25 +0000]
0026894: Intersection algorithm between two infinite cylinders is hanging

1. VRange of intersection curve has been limited. As result, too oblong intersection curve(s) will be never returned.
2. Now, purger algorithm is not called for lines obtained by Geom-Geom intersection method.
3. New statuses are entered in IntPatch_ImpImpIntersection class. It makes intersection algorithm more informative and flexible for using.
4. Method IntPatch_ImpImpIntersection::GetStatus() has been created.

Tuning of test case bugs modalg_6/bug26894

3 years ago0023178: Intersection of cylinders fails to produce results
nbv [Fri, 23 Sep 2016 14:24:52 +0000]
0023178: Intersection of cylinders fails to produce results

1. Unification of trimmed and not-trimmed cylinders processing (IntPatch_Intersection::GeomGeomPerfomTrimSurf() method has been removed).
2. Interface of IntPatch_ImpImpIntersection::Perform(...) method has been changed.
3. Now, WLine purging is forbidden for Geom-Geom-Intersection.
4. Bnd_Range class has been created. See Bnd_Range.hxx for detail information.
5. Algorithm of AddBoundaryPoint function has been improved in order to obtain intersection points in both boundaries (VFirst and VLast of every surface).
6. Earlier, method Geom2dConvert::ConcatG1(...) increased resulted B-spline degree (in case of not succession of previous iteration). Now increased value has been limited by Geom2d_BSplineCurve::MaxDegree() value (max degree = 25).
7. Algorithm of B-spline closure definition has been changed in the methods Geom2dConvert::C0BSplineToC1BSplineCurve(...) and Geom2dConvert::C0BSplineToArrayOfC1BSplineCurve(...).

Creation of test case for this issue.
Adjusting test cases according to their new behavior.

Small correction in the code according to KGV's remark.

3 years ago0027864: Add vtk support to products java wrapper
apn [Thu, 6 Oct 2016 09:36:21 +0000]
0027864: Add vtk support to products java wrapper

Fix redefinition IVtkVTK_EXPORT error during wrapping in products java sample.
Removed defined(__WIN32__)

3 years ago0027905: Visualization - randomly directed arrows on test v3d mesh B7
ski [Thu, 6 Oct 2016 09:30:39 +0000]
0027905: Visualization - randomly directed arrows on test v3d mesh B7

Behaviour of command meshvectors is stable now.
It shows default vector (0,0,1) for volume entity.
Test cases for issue CR27905

3 years ago0027929: Methods D0 and D1 for trimmed offset surface return different values if...
nbv [Wed, 5 Oct 2016 11:36:33 +0000]
0027929: Methods D0 and D1 for trimmed offset surface return different values if the surface has osculating surface

Using osculating surface is forbidden if the normal to basis (for offset) surface is well defined.

Creation of test case for this issue.
Adjusting test cases according to their new behavior.

3 years ago0027892: Construction error in offset
nbv [Tue, 27 Sep 2016 14:46:14 +0000]
0027892: Construction error in offset

Provide proper initialization of the field Geom_OffsetSurface::myOscSurf in the case of creating offset of the trimmed bspline surface.

Adjusting test cases according to their new behavior.

3 years ago0024073: Coding rules - ambiguous call to overloaded function StdPrs_DeflectionCurve...
kgv [Mon, 3 Oct 2016 11:41:12 +0000]
0024073: Coding rules - ambiguous call to overloaded function StdPrs_DeflectionCurve::Match()

Default parameters values have been removed from the methods
StdPrs_DeflectionCurve::Match() to avoid ambiguity.

3 years ago0027899: Visualization, Ray Tracing - Provide ability to rebuild GLSL shaders on...
dbp [Sat, 24 Sep 2016 17:26:33 +0000]
0027899: Visualization, Ray Tracing - Provide ability to rebuild GLSL shaders on the fly

For debugging and performance analysis it is reasonable to provide the ability to rebuild ray tracing shaders on the fly.
In this way, it will be possible to analyze the impact of different shaders modifications
without re-launching and re-configuring the scene.
This functionality can be used in the following way:
> vrenderparams -rebuild

3 years ago0027916: Visualization - access violation occurs within AIS_ColoredShape::Compute...
kgv [Wed, 28 Sep 2016 17:18:25 +0000]
0027916: Visualization - access violation occurs within AIS_ColoredShape::Compute() for NULL shape

AIS_ColoredShape::Compute() - add NULL shape check.
AIS presentations - removed redundant clearance within ::Compute()
which is always done in advance within PrsMgr_PresentationManager::Update().

3 years ago0027883: Visualization - AIS_InteractiveContext::IsDisplayed() does not return displa... IR-2016-09-30
kgv [Mon, 26 Sep 2016 08:38:18 +0000]
0027883: Visualization - AIS_InteractiveContext::IsDisplayed() does not return displayed state for specified Display Mode

3 years ago0027890: BndLib_Add2dCurve::Add(..) works incorrect on some curves
isn [Thu, 22 Sep 2016 11:42:30 +0000]
0027890: BndLib_Add2dCurve::Add(..) works incorrect on some curves

Draw Harness command 'gbounding' has been extended
to pass curve2d as args and to work in two modes ('normal' and optimal).

3 years ago0027888: Fuse of valid untouched solids leads to result with faulties
msv [Wed, 21 Sep 2016 08:02:25 +0000]
0027888: Fuse of valid untouched solids leads to result with faulties

Incorrect result of classification of a point relatively a solid leads to faulty interferences between vertices/edges of one solid with another one.  Classification has been corrected by checking if an auxiliary point in face coincides with the face boundary in 3D space.

Test case has been created.

3 years ago0027713: Test bugs fclasses bug6143 is not OK for Linux after restoring FPS (26329)
abv [Mon, 29 Aug 2016 03:22:44 +0000]
0027713: Test bugs fclasses bug6143 is not OK for Linux after restoring FPS (26329)

Use standard (POSIX-compliant) macro "__linux__" for identification of Linux system, instead of obsolete "linux"

3 years ago0026329: Restore floating point signals handling in DRAW
ifv [Tue, 9 Aug 2016 17:12:59 +0000]
0026329: Restore floating point signals handling in DRAW

Added DRAW command dsetsignal, resetting OSD signal handler with either armed or disabled FPE handler, according to an option.
If called without arguments, it sets FPE handler only if environment variable OSD_FPE is defined (with value different from 0).
On start, DRAW calls dsetsignal to set FPE signal if CSF_FPE is defined.
Test bugs fclasses bug6143 uses dsetsignal to set FPE handler unconditionally before the test command, and resets it to default at the end.

A number of changes in the code have been done in order to fix floating point exceptions that became generated after enabling signals:

- Global functions Sinh() and Cosh() defined in Standard_Real.hxx are improved to raise Standard_NumericError exception if argument is too big (greater than 710.47586), instead of relying on system treatment of floating point overflow. These functions are used instead of sinh and cosh in ElCLib.cxx.

- Maximal value of parameter on hyperbola is restricted by 23 (corresponding to ~1e10 in 3d) in order to avoid FP overflow in Extrema_GenExtCS.cxx, ShapeFix_EdgeProjAux.cxx.

- Interface of the root curve adaptor class Adaptor3d_Curve has been updated to add new virtual methods BasisCurve and OffsetValue. They complement the adaptor for the case of offset curves. These methods are used in Extrema_GenExtCS.cxx to restrict domain search in the case of offset of hyperbola, in order to get rid of floating point overflow. All classes inheriting Adaptor3d_Curve have been changed to implement the new virtual methods.

- Protection against division by zero has been implemented in ApproxInt_KnotTools.cxx, BRepClass3d_SClassifier.cxx, BRepGProp_Face.cxx, BRepMesh_FastDiscretFace.cxx, Geom2dGcc_Circ2d2TanOnIter.cxx, Geom2dInt_Geom2dCurveTool.cxx, IntPolyh_MaillageAffinage.cxx.

- Protection against calling of math functions of infinite arguments has been added in BRepCheck_Edge.cxx, BRepLib.cxx, CSLib_NormalPolyDef.cxx, Extrema_FuncExtPC.gxx, Extrema_GExtPC.gxx, Extrema_GLocateExtPC.gxx, Intf_InterferencePolygonPolyhedron.gxx, ShapeAnalysis_Surface.cxx, ShapeAnalysis_TransferParametersProj.cxx, ShapeAnalysis_Wire.cxx, math_FunctionSetRoot.cxx.

- Proper initialization of local variables is done in BOPAlgo_PaveFiller_6.cxx, XSDRAWSTLVRML.cxx.

- Inconsistent usage of Standard_Boolean* to access integer data in HLR (caused by #27772) is corrected

Some test cases have been updated to actual state.

3 years ago0027901: Resource_Manager creates directory with insufficient permissions
abv [Sun, 25 Sep 2016 20:05:00 +0000]
0027901: Resource_Manager creates directory with insufficient permissions

Resource_Manager now sets RWXD permissions for the created directories to allow their removal.
Method OSD_Directory::Build() will create intermediate directories if they do not exist yet.
OSD_FileNode is corrected to clean error status in method Exists().

Tests bugs fclasses bug181_1 and bug181_2 are corrected to check existence of generated resource files; TODO is removed in bug181_2.

3 years ago0027794: A danger in the 'NCollection_Vector' initializing constructor
abv [Mon, 26 Sep 2016 07:32:06 +0000]
0027794: A danger in the 'NCollection_Vector' initializing constructor

Constructor of NCollection_Vector is now declared as explicit.

3 years ago0027893: Visualization - AIS_InteractiveContext::SetSelected does not work
vpa [Mon, 26 Sep 2016 17:34:26 +0000]
0027893: Visualization - AIS_InteractiveContext::SetSelected does not work

- misprint in check of global selection existence in AIS_InteractiveContext::SetSelected was corrected;
- remaining occurrencies of hard-coded 0 selection mode were corrected to use global selection mode;
- test case for issue #27983

3 years ago0027891: Visualization - FPE due to use of DBL_MAX in SelectMgr_SelectingVolumeManage...
kgv [Fri, 23 Sep 2016 12:33:40 +0000]
0027891: Visualization - FPE due to use of DBL_MAX in SelectMgr_SelectingVolumeManager::DetectedPoint()

3 years ago0027818: Visualization - provide an interface to define highlight presentation properties
vpa [Wed, 28 Sep 2016 09:43:17 +0000]
0027818: Visualization - provide an interface to define highlight presentation properties

- introduces a wrapper for setting up highlight properties: Graphic3d_HighlightStyle;
- API of all methods that use highlight or selection color is changed to deal with Graphic3d_HighlightStyle;
- highlight in shading mode now supports transparency, implemented via blending;
- transparency for selection can also be set, but implementing custom entity owners with additional presentation on application level;
- methods PrsMgr_PresentationManager::Highlight, PrsMgr_PresentationManager::BoundBox that highlight object with hard-coded color are removed;
- deprecated methods of SelectMgr_EntityOwner, that use presentation manager's highlight method, were removed;
- methods of IsHilighted AIS context with selection color checks were replaced;
- added API to store dynamic and selection highlight to Prs3d_Drawer class;
- customization of dynamic and selection highlight for particular objects is now available through SelectMgr_SelectableObject::HilightAttributes();
- AIS_InteractiveContext highlight methods were updated to support individual highlight styles of interactive objects;
- introduced new command - vselprops, that allows to customize global selection and highlight properties like autoactivation, pixel tolerance and colors;
- Draw Harness commands vautoactivatesel and vselprecision were removed, use vselprops instead;
- fixed bug in command's parser;
- test case for issue #27818

3 years ago0027607: Visualization - Implement adaptive screen space sampling in path tracing
dbp [Wed, 13 Jul 2016 09:19:27 +0000]
0027607: Visualization - Implement adaptive screen space sampling in path tracing

This commit provides useful functionality for path tracing rendering core.

1) Graphic3d_RenderingParams class was extended with additional AdaptiveScreenSampling option (disabled by default).
   If this option is enabled, path tracing tries to adjust the number of samples for different screen areas.

   In this way, the more complex areas (from the point of light conditions) are sampled more intensively,
   while the simple areas are sampled very rarely.
   For example, caustics and glossy reflections are typical candidates for more precise sampling.

   In general, this allows to equalize image convergence and not to waste resources for already converged areas.
   It is also possible to visualize sampling densities by enabling ShowSamplingTiles option
   (activating and deactivating this option does not affect on the accumulated image).

2) Mixing OpenGL and ray-tracing output has been changed.
   Now blending is performed using OpenGL functionality, while ray-tracing shaders only output correct Z-value.

Test case bugs vis bug27083 has been updated,
since the alpha value is now correctly set by Ray-Tracing to 1, opaque.

3 years ago0027900: Coding rules - drop redundant Name parameter from V3d_Viewer constructor
kgv [Sun, 25 Sep 2016 16:09:23 +0000]
0027900: Coding rules - drop redundant Name parameter from V3d_Viewer constructor

Removed unused methods AIS_InteractiveContext::DomainOfMainViewer(),
V3d_Viewer::NextName(), V3d_Viewer::Domain().

NCollection_List - added methods Contains() and Remove() taking item value.
V3d_Viewer now uses V3d_ListOfLight, V3d_ListOfView instead of V3d_ListOfTransient.

Removed obsolete ZFit workaround from command XShow.

3 years ago0025180: Visualization - Homogeneous transformation API in TKV3d
kgv [Sat, 17 Sep 2016 16:33:53 +0000]
0025180: Visualization - Homogeneous transformation API in TKV3d

PrsMgr_PresentableObject, Graphic3d_Structure now consistently
take and store Handle(Geom_Transformation) instead of
TColStd_Array2OfReal / Graphic3d_Mat4.
Low-level advanced methods have been modified to pass Handle(Geom_Transformation).
High-level methods have been preserved accepting old syntax taking gp_Trsf.

Geom_Transformation now inlines most methods.

3 years ago0027860: Visualization - clean up Transformation Persistence API
kgv [Mon, 19 Sep 2016 20:40:07 +0000]
0027860: Visualization - clean up Transformation Persistence API

Graphic3d_Camera::TransformMatrices redundant NCollection_Handle usage
has been replaced with validity flags.

Graphic3d_TransModeFlags now defined as enumeration, not integer bit flags.
Graphic3d_TMF_PanPers and Graphic3d_TMF_FullPers have been removed.
Graphic3d_TMF_ZoomRotatePers has been introduced.

Graphic3d_TransformPers is now inherits Standard_Transient.
Graphic3d_TransformPers now defines dedicated constructors
for 3D persistence (zoom / rotate) and 2D persistence (2d / trihedron).
2D persistence now supports dedicated values for X and Y offsets.
The corner is now specified by enumeration Aspect_TypeOfTriedronPosition
instead of indirect interpretation of anchor point values.
Fixed handling of Graphic3d_TMF_ZoomRotatePers (combination of Graphic3d_TMF_RotatePers + Graphic3d_TMF_ZoomPers).

PrsMgr_PresentableObject, Graphic3d_CStructure now hold
Handle(Graphic3d_TransformPers) instead of a value.
Method ::SetTransformPersistence(), ::TransformPersistence()
now works with Handle(Graphic3d_TransformPers).
Old methods have been marked deprecated.

3 years ago0027696: Return max distance in xdistcs Draw command
aml [Thu, 22 Sep 2016 05:12:16 +0000]
0027696: Return max distance in xdistcs Draw command

Message about maximal distance is added for xdistcs command.

3 years ago0027873: Exception is raised in BRepFill_Filling::FindExtremitiesOfHoles()
nbv [Thu, 15 Sep 2016 14:53:06 +0000]
0027873: Exception is raised in BRepFill_Filling::FindExtremitiesOfHoles()

The reason of exception has been eliminated.
Creation of the test case for this issue.
Correction of unstable test case.

Some test cases have been adjusted according to their new behavior.

3 years ago0027879: Draw Harness, ViewerTest - disable VSync within non-interactive mode
kgv [Sat, 17 Sep 2016 10:32:13 +0000]
0027879: Draw Harness, ViewerTest - disable VSync within non-interactive mode

3 years ago0027720: HLRBrep_Algo BSpline missing edges
jgv [Thu, 8 Sep 2016 15:20:57 +0000]
0027720: HLRBrep_Algo BSpline missing edges

    The algorithm that builds outlines ("silhouettes") makes an outline in 2d parametric space of the surface starting from some previously detected point where normal is orthogonal to direction of view. So, the surface is previously discretized into (m*n) sample points and some of them become starting points for future outlines.

    If the surface has non-uniform parametrization and/or some local extremums of curvature, the outlines can not be built without breaks, so there are several groups of consequent outlines in this case. Unfortunately, it leads to the situation when current number of sample points becomes insufficient to build all the parts of outlines.

    The idea is to detect the "holes" between already constructed parts of outlines and then complete the construction.

New auxiliary draw command for testing of HLR.

Correction according to the remarks.

Update of test case according to the developer's directive

3 years ago0006768: Problems making offset wires
bugmaster [Fri, 16 Sep 2016 11:49:16 +0000]
0006768: Problems making offset wires

Adding testing case

3 years ago0027880: Samples - fix handling of Unicode paths within MFC import/export sample
kgv [Mon, 19 Sep 2016 08:46:27 +0000]
0027880: Samples - fix handling of Unicode paths within MFC import/export sample

3 years ago0014531: Boolean Operation Algorithm fails
bugmaster [Wed, 21 Sep 2016 07:58:11 +0000]
0014531: Boolean Operation Algorithm fails

Adding test case

3 years ago0027875: GeomFill_NSections constructor crash on sequence of curve containing only...
nbv [Fri, 16 Sep 2016 12:31:29 +0000]
0027875: GeomFill_NSections constructor crash on sequence of curve containing only one curve

The GeomFill_NSections algorithm cannot create any surface from sequence with single curve only.

Therefore, return is provided in corresponding place of the code. Additionally, some public methods of GeomFill_NSections class checks if the surface has been created earlier.

Test case for this issue has been created.

Compiler error has been eliminated.

3 years ago0027822: Exception access violation is raised in BRepOffsetAPI_MakePipeShell during...
jgv [Mon, 19 Sep 2016 10:35:27 +0000]
0027822: Exception access violation is raised in BRepOffsetAPI_MakePipeShell during of build

New public method IsDone() is added to abstract class BRepFill_SectionLaw.

Test case bugs/modlag_6/bug27822 has been added

3 years ago0010234: BRepOffsetAPI_MakePipeShell fails on spine built from two segments and profi...
bugmaster [Mon, 19 Sep 2016 12:03:00 +0000]
0010234: BRepOffsetAPI_MakePipeShell fails on spine built from two segments and profile built from arc

Adding testing case

3 years ago0027567: VIS - possible memory leaks due to use of plain pointers: Fix also the VIS...
rkv [Tue, 23 Aug 2016 12:52:11 +0000]
0027567: VIS - possible memory leaks due to use of plain pointers: Fix also the VIS guide. Add the test v3d/ivtk/bug27567. Add a draw command "ivtkremove".
0027734: Configuration - TKIVtkDraw build fails with TBB: Remove unnecessary define statement (windows specific).

Small correction of test case for issue CR27567

3 years ago0027838: Foundation Classes - support wchar_t* input within TCollection_AsciiString... IR-2016-09-15
kgv [Sun, 4 Sep 2016 11:31:47 +0000]
0027838: Foundation Classes - support wchar_t* input within TCollection_AsciiString and TCollection_ExtendedString

TCollection_ExtendedString/TCollection_AsciiString description
has been updated to reflect usage of this classes for Unicode strings.

TCollection_ExtendedString now defines constructor taking wchar_t* (all platforms)
and method ::ToWideString() returning wchar_t* (Windows only).
TCollection_AsciiString now defines constructor taking wchar_t*.

TCollection_ExtendedString/TCollection_AsciiString now defines
auxiliary methods ::StartsWith() and ::EndsWith().

TCollection_ExtendedString internals has been updated to eliminate
duplicated code for converting between UTF-16 and UTF-8.

Code has been cleaned up from redundant explicit conversions to wchar_t*.
Global method OSD_OpenStream()/OSD_OpenFileBuf() have been replaced
by C++ template to eliminate copy-paste for different STL collections.

OSD_SharedLibrary now uses wide-char system API call LoadLibraryExW()
on Windows for consistency.

New macro Standard_UNUSED has been added for marking possibly unused functions and variables
(to suppress gcc/clang compiler warnings).

3 years ago0027808: Some geometric_tolerances are not imported.
ika [Tue, 13 Sep 2016 07:47:23 +0000]
0027808: Some geometric_tolerances are not imported.

Process datum_feature entity during collecting shape_aspects of GDT.
Update test cases.

3 years ago0027830: Infinite HLR looping
msv [Thu, 8 Sep 2016 19:57:50 +0000]
0027830: Infinite HLR looping

Make protection of HLR algo against garbage data in faces. In particular case, there are faces built on a periodical surfaces, which U bounds exceed period thousands times. Such faces are excluded from the process of edges hiding.

In addition, while fitting the intersection point in period for periodical faces, replace looping with the single call to AdjustPeriodic method.

- Add new test case.
- Update tests of HLR according to new numbers of subshapes.

Update of test cases according to the new behavior

3 years ago0027867: Visualization - AIS_InteractiveContext::HasSelectedShape() returns true...
Sergey Solomin [Tue, 13 Sep 2016 08:07:03 +0000]
0027867: Visualization - AIS_InteractiveContext::HasSelectedShape() returns true if selected interactive is instance of AIS_Shape only

3 years ago0027861: Configuration - include ApplicationFramework to UWP list
kgv [Sat, 10 Sep 2016 12:38:05 +0000]
0027861: Configuration - include ApplicationFramework to UWP list

3 years ago0022037: Exception (Construction Error) during splitting face with hole by another...
bugmaster [Wed, 14 Sep 2016 11:22:59 +0000]
0022037: Exception (Construction Error) during splitting face with hole by another face

Adding test case

3 years ago0025138: SIGSEGV when sweeping along helix
bugmaster [Wed, 14 Sep 2016 09:20:37 +0000]
0025138: SIGSEGV when sweeping along helix

Adding test case

3 years ago0027816: Visualization - provide an API for overriding clipping planes list
kgv [Wed, 14 Sep 2016 10:44:47 +0000]
0027816: Visualization - provide an API for overriding clipping planes list

Graphic3d_SequenceOfHClipPlane now inherits Standard_Transient.
PrsMgr_PresentableObject, Graphic3d_Structure, Graphic3d_CStructure,
V3d_View, OpenGl_View now manages the plane list by Handle.
The getters ::GetClipPlanes() has been removed,
setters taking non-handle ::SetClipPlanes() has been marked deprecated.

OpenGl_Structure::Render() and SelectMgr_ViewerSelector::checkOverlap()
now disable global (view) clipping planes for objects
with flags Graphic3d_TMF_TriedronPers and Graphic3d_TMF_2d
or with new flag Graphic3d_SequenceOfHClipPlane::ToOverrideGlobal().

OpenGl_Clipping now implements interface for managing clipping planes
without copying the sequences.
The filtering of duplicates is no more performed by OpenGl_Clipping
- application is responsible to not do this.
OpenGl_Clipping tries avoiding unnecessary allocations for managing
list of active planes.

MFC sample has been updated to use V3d_View::ClipPlanes() method.

3 years ago0027862: Exception in BRepOffsetAPI_MakePipeShell
msv [Mon, 12 Sep 2016 07:21:53 +0000]
0027862: Exception in BRepOffsetAPI_MakePipeShell

Check if the shape is not null before querying its ShapeType.
Create test case for the bug.

Small correction of test case for issue CR27862

3 years ago0027854: Foundation Classes - OSD_Parallel::NbLogicalProcessors() should handle Andro...
kgv [Thu, 8 Sep 2016 19:54:54 +0000]
0027854: Foundation Classes - OSD_Parallel::NbLogicalProcessors() should handle Android specifically

Read CPU info from sysfs.

3 years ago0027844: Foundation Classes - remove method OSD_File::Print()
kgv [Mon, 12 Sep 2016 06:54:55 +0000]
0027844: Foundation Classes - remove method OSD_File::Print()

3 years ago0027853: Visualization, OpenGl_Texture - optimize sequential upload of texture image
kgv [Thu, 8 Sep 2016 14:16:56 +0000]
0027853: Visualization, OpenGl_Texture - optimize sequential upload of texture image

Graphic3d_TextureRoot - added methods Revision() and UpdateRevision()
for marking updates in texture data source.
OpenGl_AspectFace handles Graphic3d_TextureRoot::Revision() changes.
OpenGl_Texture::Init() now patches already allocated texture image when possible.

Graphic3d_Texture2D - added methods HasMipMaps()/SetMipMaps()
for configuring MipMap usage (as alternative to sub-classing).

3 years ago0027851: Wrong result of classification of the point relative to the solid
nbv [Mon, 12 Sep 2016 14:48:59 +0000]
0027851: Wrong result of classification of the point relative to the solid

Test case creation.

3 years ago0027863: Geom_BSplineSurface::SetVKnot(const Standard_Integer, const Standard_Real...
razmyslovich [Mon, 12 Sep 2016 09:17:48 +0000]
0027863: Geom_BSplineSurface::SetVKnot(const Standard_Integer, const Standard_Real) has no effect

Fix Geom_BSplineSurface::SetVKnot function to have the same behavior as SetUKnot.

3 years ago0027739: Visualization, TKV3d - implement individual acceleration data structure...
apl [Fri, 9 Sep 2016 12:53:58 +0000]
0027739: Visualization, TKV3d - implement individual acceleration data structure for selection of 2D persistent objects

Low-level selection algorithms has been improved to explicitly support 2D
transformation persistent objects. This patch generally touches:

1) SelectMgr_SelectableObjectSet - the lists of objects has been split onto three
subsets (regular, 3d-persistent, 2d-persistent). Each subset has individual BVH
tree. The algorithms are now have more flexibility to update only those trees
that is really required for actual camera state.

2) SelectMgr_ViewerSelector - explicitly supports Eye space selection operations
on BVH tree for 2d-persistent subset. Change of camera position does not
anymore affect acceleration data structure (BVH tree) of 2d-persistent selectable objects.

3) Other classes from SelectMgr have been fine-tuned to provide appropriate API.

Porting notes:

This patch touches very low-level selection classes. If the low-level features were used
the following modifications may need to be considered for porting:

1) Iteration over objects of SelectMgr_SelectableObjectSet should now
be implemented with corresponding Iterator class.

2) SelectMgr_BaseFrustum::ScaleAndTransform and derived classes return
Handle(SelectMgr_BaseFrustum) instead of NCollection_Handle<> type.

Small correction of test case for issue CR27739

3 years ago0027313: Exception during WriteStep with PMI
ika [Tue, 28 Jun 2016 09:00:13 +0000]
0027313: Exception during WriteStep with PMI

Add check for datum position during export of datum_system
Add Draw commands for set/get datum position
update test cases

Correction of case bugs/step/bug27313

3 years ago0025423: Incorrect result of splitting some shapes
bugmaster [Fri, 9 Sep 2016 13:10:56 +0000]
0025423: Incorrect result of splitting some shapes

Adding testing case

3 years ago0022609: Cut operation between two solids produces not valid shape
bugmaster [Thu, 8 Sep 2016 13:22:40 +0000]
0022609: Cut operation between two solids produces not valid shape

Adding test case

3 years ago0022454: Export /Import of valid model leads to lost of tolerances
bugmaster [Thu, 8 Sep 2016 15:19:41 +0000]
0022454: Export /Import of valid model leads to lost of tolerances

Adding testing case

3 years ago0027842: Exception in intersection algorithm if FPE is switched on IR-2016-09-08
nbv [Tue, 6 Sep 2016 12:35:35 +0000]
0027842: Exception in intersection algorithm if FPE is switched on

1. The reason of exception has been eliminated.
2. Interfaces of DistanceMinimizeByGradient and DistanceMinimizeByExtrema methods of IntWalk_PWalking class has been changed.

Creation of test case for this issue.

TODO has been added with reference to the issue #26329

3 years ago0027836: Visualization, TKOpenGl - gradient background is lost at some camera positions
kgv [Sat, 3 Sep 2016 18:37:55 +0000]
0027836: Visualization, TKOpenGl - gradient background is lost at some camera positions

OpenGl_View::ZLayerBoundingBox() now adds screen background plane for proper Z-fit.

Graphic3d_TransformPers::Compute() - projection matrix has been excluded
from the math to eliminate floating point computation error.

3 years ago0027827: Samples - fix compilation errors in XAML (UWP) sample.
ski [Wed, 31 Aug 2016 09:35:03 +0000]
0027827: Samples - fix compilation errors in XAML (UWP) sample.

Error about deployment target version was fixed.
Check of export to VRML format was added to sample XAML (UWP).

Add freetype.dll to the sample project for correct execution of sample from Visual Studio.

3 years ago0025152: Incorrect result of Boolean Union by BRepAlgoAPI_Fuse
bugmaster [Wed, 7 Sep 2016 12:11:32 +0000]
0025152: Incorrect result of Boolean Union by BRepAlgoAPI_Fuse

Adding testing case

3 years ago0025102: A part of intersection cannot be found
bugmaster [Wed, 7 Sep 2016 10:28:59 +0000]
0025102: A part of intersection cannot be found

Adding testing case

3 years ago0027222: Empty result of making volume operation
emv [Tue, 30 Aug 2016 05:36:48 +0000]
0027222: Empty result of making volume operation

1. BOPAlgo_MakerVolume - Taking into account the possibility of Same Domain faces
   while collecting the faces for building the solids;
2. BOPDS_DS - Making the check on coincidence of edges consistent with the intersection
   algorithm which also adds the Precision::Confusion() to the intersection tolerance;
3. Test case for the issue.
4. Adjusting test cases (improvements) to their new behavior.

3 years ago0027805: Visualization - AIS_InteractiveContext::FitSelected() is broken for global...
kgv [Sat, 3 Sep 2016 09:23:55 +0000]
0027805: Visualization - AIS_InteractiveContext::FitSelected() is broken for global selection

AIS_Selection class has been cleaned up:
- SelectMgr_EntityOwner is now used in interface instead of Standard_Transient.
- Method ::Select() has been renamed to ::Clear() for clarity.
- Dropped unused class fields.
- Owners without Selectable are not added to the list (should never happen).

SelectMgr_SelectableObject::BndBoxOfSelected() has been modified
to properly compute bounding box of ALL selection Entities
for specified Owner (only first Entity has been used before the patch).

Methods of AIS_InteractiveContext::DisplaySelected(), ::SetSelectedAspect(),
::FitSelected(), ::SubIntensityOff() have been fixed (wrong DownCast).
AIS_InteractiveContext::AddSelect() now handles properly AIS_InteractiveObject as argument.

3 years ago0027804: Two breps cause intersections to loop for too long/infinitely
msv [Fri, 26 Aug 2016 14:33:46 +0000]
0027804: Two breps cause intersections to loop for too long/infinitely

In ProjLib_ComputeApprox algorithm, compute correct parametric tolerance from the input 3D tolerance using surface resolution, in order to pass it to low-level 2D algorithm Approx_FitAndDivide2d (instantiation of the generic class Approx_ComputeCLine). Earlier 3D tolerance was used as parametric tolerance directly, which was a problem for surfaces with too small radius of curvature.

Also, eliminate redundant creation of the object of type AppParCurves_MultiCurve on each iteration in the method Approx_ComputeCLine::Compute.

The post treatment of the Edge/Edge intersections has been improved. Namely:
a. Making the procedure of sharing Edge/Edge intersection vertices consistent with intersection algorithm by enlarging bounding box of each vertex on half of Precision::Confusion();
b. Algorithm of computation of vertex tolerance (in order to cover tangent zone between Edges) has been changed for Line/Circle cases.

Test cases for issue CR27804
Adjusting test cases according to their new behavior.

Mark the test boolean/volumemaker/A8 as unstable between different versions of MSVS (2010 and 2013).

3 years ago0027759: Invalid result of a Boolean fuse operation
emv [Tue, 30 Aug 2016 05:00:08 +0000]
0027759: Invalid result of a Boolean fuse operation

Test case for the issue.

3 years ago0025055: Incorrect shape copying in BRepBuilderAPI_Copy
bugmaster [Wed, 7 Sep 2016 07:32:49 +0000]
0025055: Incorrect shape copying in BRepBuilderAPI_Copy

Adding testing case

3 years ago0025534: TObj_Application unicode path issue.
mpv [Mon, 5 Sep 2016 10:27:40 +0000]
0025534: TObj_Application unicode path issue.

3 years ago0027835: Application Framework, BinXCAF - handle correctly faces with NULL surface...
kgv [Sat, 3 Sep 2016 16:21:58 +0000]
0027835: Application Framework, BinXCAF - handle correctly faces with NULL surface within BinTools_ShapeSet

BinTools_ShapeSet::AddGeometry() now writes NULL surface with 0 index.
myWithTriangles is ignored for TopoDS_Face with NULL surface so that
triangulation-only faces are not lost with defaults.

Added Draw Harness command StoreTriangulation defining
WithTriangulation flag for BinXCAF/BinOcaf storage drivers.

3 years ago0027815: Visualization, TKV3d - AIS_InteractiveContext has inconsistent display mode API
san [Fri, 26 Aug 2016 16:04:23 +0000]
0027815: Visualization, TKV3d - AIS_InteractiveContext has inconsistent display mode API

3 years ago0027834: Visualization, SelectMgr_ViewerSelector - iteration through detected Entitie...
kgv [Fri, 2 Sep 2016 16:11:23 +0000]
0027834: Visualization, SelectMgr_ViewerSelector - iteration through detected Entities should be sorted

SelectMgr_SortCriterion now stores detected SensitiveEntity and 3D point.
SelectMgr_ToleranceMap class definition has been moved out to dedicated file (from SelectMgr_ViewerSelector).

SelectMgr_ViewerSelector - the methods implementing class-as-iterator
Init(), More(), Next(), Picked() and InitDetected(), MoreDetected(),
NextDetected(), DetectedEntity() have been deprecated.
User should access detection results by using index.
New methods PickedData(), PickedEntity(), PickedPoint() have been added
for accessing auxiliary information about picked object in sorted order.

3 years ago0027811: Configuration - allow building TKOpenGl with OpenGL ES on Windows
kgv [Wed, 31 Aug 2016 16:34:32 +0000]
0027811: Configuration - allow building TKOpenGl with OpenGL ES on Windows

OpenGl_ShaderManager::prepareStdProgramFboBlit() now tries using extension
GL_EXT_frag_depth within OpenGL ES 2.0 when OpenGL ES 3.0 is not available.
OpenGl_View::blitBuffers() now disables Depth test
when copying depth values is not supported by OpenGL ES 2.0 hardware.

Fixed building for UWP with SDK 10.0.10240.0
Fixed building TKService, TKV3d and TKOpenGl for UWP.

OSD_Environment now defines global environment map
for emulating desktop behavior on UWP.

3 years ago0027819: Visualization - provide a possibility to redefine SelectMgr_SelectableObject...
vpa [Wed, 31 Aug 2016 14:42:50 +0000]
0027819: Visualization - provide a possibility to redefine SelectMgr_SelectableObject::UpdateSelection

- SelectMgr_SelectableObject::UpdateSelection calls redefineable SelectMgr_SelectableObject::updateSelection

3 years ago0027813: Visualization - add method V3d_View::DiagnosticInformation() similar to...
kgv [Wed, 31 Aug 2016 18:06:54 +0000]
0027813: Visualization - add method V3d_View::DiagnosticInformation() similar to vglinfo command

New method V3d_View::DiagnosticInformation() has been introduced
providing the access to low-level OpenGL context information
for diagnostic automated reports or displaying in application About System.

3 years ago0027817: Visualization, StdSelect_BRepOwner - copy transformation persistence flag IR-2016-09-01
kgv [Sun, 28 Aug 2016 16:21:13 +0000]
0027817: Visualization, StdSelect_BRepOwner - copy transformation persistence flag

3 years ago0027821: Visualization, AIS_Shape - add NULL checks for displaying TopoDS_Face with...
akz [Fri, 5 Aug 2016 08:24:58 +0000]
0027821: Visualization, AIS_Shape - add NULL checks for displaying TopoDS_Face with NULL surface NULL surface

3 years ago0027807: Units are lost during import presentation of GDT.
ika [Fri, 26 Aug 2016 13:10:19 +0000]
0027807: Units are lost during import presentation of GDT.

Add entities to import a subtype of draughting_model,
Add units calculating during STEP reading.

3 years ago0022582: Visualization - provide an API for dumping a sub-region of the viewport
osa [Sun, 21 Aug 2016 18:55:10 +0000]
0022582: Visualization - provide an API for dumping a sub-region of the viewport

Graphic3d_Camera definition has been exended by optional Tile property (Graphic3d_CameraTile structure).
V3d_View::ToPixMap() now performs tiled dump when image size exceeds hardware limits.

OpenGl_View::Redraw() - fixed dump of immediate Z layers in no stereo and no MSAA case.

OpenGl_Context now tracks viewport values.
Draw Harness command vdump has been extended with new argument -tileSize.

3 years ago0027810: Compilation error on NCollection_UBTree::Kill()
oan [Thu, 25 Aug 2016 13:46:13 +0000]
0027810: Compilation error on NCollection_UBTree::Kill()

3 years ago0027796: Visualization - allow 3D objects with Graphic3d_TMF_2d flag
kgv [Sat, 20 Aug 2016 16:37:56 +0000]
0027796: Visualization - allow 3D objects with Graphic3d_TMF_2d flag

Graphic3d_TransformPers::Apply() now does not reset projection matrix
for objects with Graphic3d_TMF_2d flag.

Useless flag Graphic3d_TMF_2d_IsTopDown has been removed.
SelectMgr_SelectableObjectTrsfPersSet now does not skip Graphic3d_TMF_2d presentations.
OpenGl_Layer::BoundingBox() now takes into account Graphic3d_TMF_2d presentations for proper Z-fit.

AIS_ColorScale now uses "lazy" mode for rendering labels
(considering 2D persistence to be already defined within entire structure).

OpenGl_Layer::updateBVH() now updates myAlwaysRenderedMap to handle
dynamic transformation persistence flag change without redisplaying the object.

3 years ago0027626: Attempt to display shape in 3d leads to very long calculation loop
apn [Wed, 6 Jul 2016 15:42:02 +0000]
0027626: Attempt to display shape in 3d leads to very long calculation loop

Added test case bugs/mesh/bug27626

Small correction of test case for issue CR27626

3 years ago0027781: [Regression to 6.9.0] Exception in ShapeFix_Shape algorithm with option...
msv [Thu, 18 Aug 2016 09:12:33 +0000]
0027781: [Regression to 6.9.0] Exception in ShapeFix_Shape algorithm with option FixSmallAreaWireMode

The matter was that during checking wires of a shape for small area, non-outer wires were checked with constructing a new face with BRepBuilderAPI_MakeFace. If a face has location an edge from original face has no pcurve on the constructed face, which leads to exception in BRepGProp::SurfaceProperties. The fix constructs new face using EmptyCopy.

The method ShapeAnalysis_Wire::CheckSmallArea() has been changed so as to check area of the outer wire without hole-wires. API of this method has been changed, as the second argument theIsOuterWire is not needed any more.

The test cases have been updated, below are explanations of changes:

test de iges_2 G7
The fixed version leaves a wire in a face, but the master version considered it small and removed. The master version works wrong. It is because the face built with this wire has negative area, but the code in CheckSmallArea function does not get absolute value before comparing area with the tolerance. The left wire leads to splitting of the face on two, checkshape error in the face, and statshape faulty due to increased number of faces in the second pass.

test de iges_2 G2
The fixed version leaves a wire that is removed in the master version. The cause is the same as in G7 test case. However, here the problematic wire has very big tolerance. So, when the fixed version left it in the shape, the overall maximal tolerance became much greater than in reference data.

test de step_3 E6
In fixed version a really bad small wire is removed from the face, while in master version it is left and produces an error in checkshape report. So, it is an improvement.

3 years ago0025220: Using BRepFill_OffsetWire withs wires containing arc edges throws exception...
bugmaster [Fri, 26 Aug 2016 12:19:19 +0000]
0025220: Using BRepFill_OffsetWire withs wires containing arc edges throws exception "Standard_NullValue"
Adding testing case

3 years ago0026910: General fuse does not find intersection between line and circle
bugmaster [Thu, 25 Aug 2016 13:57:08 +0000]
0026910: General fuse does not find intersection between line and circle
Adding testing case

3 years ago0027772: Foundation Classes - define Standard_Boolean using C++ type "bool" instead... IR-2016-08-25
kgv [Thu, 25 Aug 2016 11:58:51 +0000]
0027772: Foundation Classes - define Standard_Boolean using C++ type "bool" instead of "unsigned int"

Code has been updated to remove no-op casts and implicit casts to Standard_Boolean.

Places of inproper use of Standard_Boolean instead of Standard_Integer
have been corrected:
- Bnd_Box, Bnd_Box2d
  Bit flags are now defined as private enum
- HLRAlgo_BiPoint, HLRAlgo_EdgesBlock, HLRBRep_EdgeData, HLRBRep_FaceData
  Bit flags are now defined as enum
- HLRAlgo_EdgeStatus, HLRBRep_BiPnt2D, HLRBRep_BiPoint
  Bit flags are now defined as bool fields
- HLRAlgo_PolyData
  Bit flags are now defined as Standard_Integer
- OSD_DirectoryIterator, OSD_FileIterator
  Boolean flag is now defined as Standard_Boolean
- ShapeAnalysis_Surface::SurfaceNewton()
  now returns Standard_Integer (values 0, 1 or 3)
- ChFi2d_FilletAlgo
  now uses TColStd_SequenceOfBoolean instead of TColStd_SequenceOfInteger
  for storing boolean flags

Method IFSelect_Dispatch::PacketsCount() has been dropped from interface.

ShapeFix_Solid::Status() has been fixed to decode requested status
instead of returning integer value.

TopOpeBRepBuild_Builder1 now defines map storing Standard_Boolean values
instead of Standard_Integer.

Persistence for Standard_Boolean type has been corrected
to keep backward compatibility:
- BinMDataStd, BinTools, FSD_BinaryFile

Broken Draw Harness commands vdisplaymode and verasemode have been removed.

BRepMesh_FastDiscretFace::initDataStructure() - workaround old gcc limitations

BRepMesh_IncrementalMesh::clear() - avoid ambiguity

3 years ago0027793: Visualization - object drifts at zoom within Graphic3d_TMF_TriedronPers...
kgv [Tue, 23 Aug 2016 14:11:49 +0000]
0027793: Visualization - object drifts at zoom within Graphic3d_TMF_TriedronPers applied

Graphic3d_TransformPers now takes Graphic3d_Camera definition as argument
for methods applying transformation.

Graphic3d_TransformPers::Apply() now computes Graphic3d_TMF_TriedronPers
transformation in the following way:
- The object is moved onto Z focus distance.
- The object is expected to be defined in pixels.
- The Z coordinate on anchor point is used as offset from the view corner in pixels.
- It is now possible to define not only corners of the view, but also middle of the side.
- Graphic3d_TMF_TriedronPers now works with perspective projection.

OpenGl_LayerList::ChangeLayer() - fixed removing of the element in old ZLayer.

OpenGl_Layer::BoundingBox() now takes into account bounding box
of Graphic3d_TMF_TriedronPers presentations for Z-fit operation.

3 years ago0027350: Support for Universal Windows Platform
ski [Fri, 12 Aug 2016 15:38:48 +0000]
0027350: Support for Universal Windows Platform

- Toolchain file to configure a Visual Studio generator for a Windows 10 Universal Application was added (CMake).
- There is no support for environment variables in UWP.
- SID is not supported (were excluded).
- Windows registry is not supported (were excluded).
- Mess with usage of Unicode/ANSI was corrected.
- Added sample to check UWP functionality.
- Excluded usage of methods with Unicode characters where it is possible.
- Minor corrections to allow building OCAF (except TKVCAF) and DE (except VRML and XDE)
- Building of unsupported modules for UWP platform is off by default .
- Checking of DataExchange functionality was added to XAML (UWP) sample.
- Added information about UWP to the documentation.
- Update of results of merge with issue 27801

3 years ago0027645: Data Exchange - access violation when reading STEP AP242 file
ika [Tue, 23 Aug 2016 08:06:03 +0000]
0027645: Data Exchange - access violation when reading STEP AP242 file

Fix exceptions, add test cases.

Small correction of test cases for issue CR27645