MantisBT - Open CASCADE
View Issue Details
0028891Open CASCADE[OCCT] OCCT:Testspublic2017-07-04 16:142018-03-15 18:40
apn 
apn 
normalminor 
feedbackreopened 
ALL
 
[OCCT] 7.4.0* 
volumerender
0028891: Add environment variable CSF_TestVolumeRenderDataPath
Add special environment variable CSF_TestVolumeRenderDataPath for loading data files from special repository for VolumeRender test cases.
set 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
No tags attached.
Issue History
2017-07-04 16:14apnNew Issue
2017-07-04 16:14apnAssigned To => apn
2017-07-05 10:37gitNote Added: 0067990
2017-07-05 10:40kgvNote Added: 0067992
2017-07-05 10:48bugmasterNote Added: 0067993
2017-07-05 10:48apnTest case number => volumerender
2017-07-05 10:48apnNote Added: 0067994
2017-07-05 10:48apnAssigned Toapn => vsv
2017-07-05 10:48apnStatusnew => resolved
2017-07-05 10:48apnSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=17034#r17034
2017-07-05 11:10kgvNote Added: 0067995
2017-07-05 11:29apnNote Added: 0067998
2017-07-05 11:33apnSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=17039#r17039
2017-07-05 12:00kgvNote Added: 0068002
2017-07-05 12:00kgvAssigned Tovsv => apn
2017-07-05 12:00kgvStatusresolved => assigned
2017-07-05 12:01kgvNote Edited: 0068002bug_revision_view_page.php?bugnote_id=68002#r17041
2017-07-05 12:01kgvNote Edited: 0068002bug_revision_view_page.php?bugnote_id=68002#r17042
2017-07-24 10:09bugmasterStatusassigned => feedback
2017-07-24 10:40bugmasterStatusfeedback => closed
2017-07-24 10:40bugmasterResolutionopen => no change required
2017-07-24 10:41bugmasterStatusclosed => feedback
2017-07-24 10:41bugmasterResolutionno change required => reopened
2017-08-27 11:00bugmasterTarget Version7.2.0 => 7.3.0
2017-09-29 17:11gitNote Added: 0071009
2018-03-15 18:40kgvTarget Version7.3.0 => 7.4.0*

Notes
(0067990)
git   
2017-07-05 10:37   
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.
(0067992)
kgv   
2017-07-05 10:40   
Please provide more detailed description to this bug elaborating why existing variables can not be used.
(0067993)
bugmaster   
2017-07-05 10:48   
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
(0067994)
apn   
2017-07-05 10:48   
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.
(0067995)
kgv   
2017-07-05 11:10   
> 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?
(0067998)
apn   
2017-07-05 11:29   
Size of database is approximately 6.8GB. It's available on \\salomer1\dn62\occt_tests_data_volumerender.
(0068002)
kgv   
2017-07-05 12:00   
(edited on: 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.

(0071009)
git   
2017-09-29 17:11   
Branch CR28891 has been deleted by kgv.

SHA-1: bf61566c69e6f2b7cb65cbc0ed0aec39cd64dfd5