View Issue Details

IDProjectCategoryView StatusLast Update
0028891Open CASCADEOCCT:Testspublic2020-09-14 10:46
Reporterapn Assigned Tobugmaster  
PrioritynormalSeverityminor 
Status closedResolutionno change required 
PlatformAOSL 
Target Version7.5.0 
Summary0028891: Add environment variable CSF_TestVolumeRenderDataPath
DescriptionAdd special environment variable CSF_TestVolumeRenderDataPath for loading data files from special repository for VolumeRender test cases.
Steps To Reproduceset env(CSF_TestVolumeRenderDataPath) /dn62/occt_tests_data_volumerender (lin)
set env(CSF_TestVolumeRenderDataPath) \\testex3\occt_tests_data_volumerender (wnt)
set env(CSF_TestScriptsPath) ./tests
testgrid volumerender
TagsNo tags attached.
Test case numbervolumerender

Activities

git

2017-07-05 10:37

administrator   ~0067990

Branch CR28891 has been created by apn.

SHA-1: bf61566c69e6f2b7cb65cbc0ed0aec39cd64dfd5


Detailed log of new commits:

Author: apn
Date: Wed Jul 5 10:36:54 2017 +0300

    0028891: Add environment variable CSF_TestVolumeRenderDataPath
    
    Add environment variable CSF_TestVolumeRenderDataPath for loading data files from special repository for VolumeRender products test cases.

kgv

2017-07-05 10:40

developer   ~0067992

Please provide more detailed description to this bug elaborating why existing variables can not be used.

bugmaster

2017-07-05 10:48

administrator   ~0067993

Usage of CSF_TestDataPath variable for definition path at volumerender shapes in OCCT testing system is not acceptable.
It cause terrible slowdown in the performance of the testing due to too long a shape search in a sharply increased database.

Creation of new variable was agreed with ABV

apn

2017-07-05 10:48

administrator   ~0067994

Environment variable CSF_TestVolumeRenderDataPath was added. I checked running volumerender test cases on CR25464-CR25464-VSV on linux, but there are some errors.

Dear vsv, could you please take commit from CR28891 and check test cases.

kgv

2017-07-05 11:10

developer   ~0067995

> Creation of new variable was agreed with ABV
I'm not against the patch, I just would like the reasons and problem description being elaborated to foresee possible issues in future.

> Usage of CSF_TestDataPath variable for definition path at volumerender shapes in OCCT testing system is not acceptable.
> It cause terrible slowdown in the performance of the testing due to too long a shape search in a sharply increased database.
So the problem is a large amount of files added for testing Volume Rendering?
How many? How large this database in size?

apn

2017-07-05 11:29

administrator   ~0067998

Size of database is approximately 6.8GB. It's available on \\salomer1\dn62\occt_tests_data_volumerender.

kgv

2017-07-05 12:00

developer   ~0068002

Last edited: 2017-07-05 12:01

> Size of database is approximately 6.8GB.
> It's available on \\salomer1\dn62\occt_tests_data_volumerender.
The problem is not specific to the Volume Renderer data set - other packages might also operate with thousands of files placed into sub-folder(s).

Example - STEP/JT assemblies with external references.
Note that it would be better skipping subfolders containing external references if not for a performance reasons,
but just to avoid possible name collision issues (it is not possible easily renaming names of external references files).

Thus, I propose more general solution to the problem - marking folders which are not expected to be recursively traversed.
This can be done by creating an empty file in the folder, e.g. "norecursion.rules" and processing such folders specifically within "locate_file" procedure.

git

2017-09-29 17:11

administrator   ~0071009

Branch CR28891 has been deleted by kgv.

SHA-1: bf61566c69e6f2b7cb65cbc0ed0aec39cd64dfd5

kgv

2020-09-11 17:10

developer   ~0094485

I propose closing this bug, as Volume Rendering component is now testable.

Issue History

Date Modified Username Field Change
2017-07-04 16:14 apn New Issue
2017-07-04 16:14 apn Assigned To => apn
2017-07-05 10:37 git Note Added: 0067990
2017-07-05 10:40 kgv Note Added: 0067992
2017-07-05 10:48 bugmaster Note Added: 0067993
2017-07-05 10:48 apn Test case number => volumerender
2017-07-05 10:48 apn Note Added: 0067994
2017-07-05 10:48 apn Assigned To apn => vsv
2017-07-05 10:48 apn Status new => resolved
2017-07-05 10:48 apn Steps to Reproduce Updated
2017-07-05 11:10 kgv Note Added: 0067995
2017-07-05 11:29 apn Note Added: 0067998
2017-07-05 11:33 apn Steps to Reproduce Updated
2017-07-05 12:00 kgv Note Added: 0068002
2017-07-05 12:00 kgv Assigned To vsv => apn
2017-07-05 12:00 kgv Status resolved => assigned
2017-07-05 12:01 kgv Note Edited: 0068002
2017-07-05 12:01 kgv Note Edited: 0068002
2017-07-24 10:09 bugmaster Status assigned => feedback
2017-07-24 10:40 bugmaster Status feedback => closed
2017-07-24 10:40 bugmaster Resolution open => no change required
2017-07-24 10:41 bugmaster Status closed => feedback
2017-07-24 10:41 bugmaster Resolution no change required => reopened
2017-08-27 11:00 bugmaster Target Version 7.2.0 => 7.3.0
2017-09-29 17:11 git Note Added: 0071009
2018-03-15 18:40 kgv Target Version 7.3.0 => 7.4.0
2019-07-19 14:35 apn Target Version 7.4.0 => 7.5.0
2020-09-11 17:01 utverdov Assigned To apn => inv
2020-09-11 17:05 utverdov Target Version 7.5.0 => 7.6.0
2020-09-11 17:09 bugmaster Assigned To inv => bugmaster
2020-09-11 17:10 kgv Note Added: 0094485
2020-09-11 17:10 kgv Target Version 7.6.0 => 7.5.0
2020-09-14 10:46 bugmaster Status feedback => closed
2020-09-14 10:46 bugmaster Resolution reopened => no change required