mirror of
https://github.com/OPM/ResInsight.git
synced 2026-10-29 16:40:20 -05:00
Do not convert full range string to wildcard in file set definition
Use of wildcard requires a file search and is slower than using the range string
This commit is contained in:
1 parent
de35c760c0
commit
213be04700
1 file changed
+6
-10
@@ -142,16 +142,12 @@ void RimEnsembleFileSet::findAndSetPathPatternAndRangeString( const QStringList&
|
||||
m_pathPattern = noExtension;
|
||||
m_realizationNumbersReadFromFiles = rangeString;
|
||||
|
||||
auto createdPaths = RiaEnsembleImportTools::createPathsFromPattern( pattern, rangeString, internal::placeholderString() );
|
||||
if ( createdPaths.size() == filePaths.size() )
|
||||
{
|
||||
// If incoming files count matches the generated file count, use '*' to represent all
|
||||
m_realizationSubSet = internal::placeholderString();
|
||||
}
|
||||
else
|
||||
{
|
||||
m_realizationSubSet = rangeString;
|
||||
}
|
||||
// When all files in an ensemble are selected, we could get the same behaviour if set the range string to a wildcard. Avoid this now, as
|
||||
// the performance is worse using a wildcard.
|
||||
// https://github.com/OPM/ResInsight/issues/12660
|
||||
// https://github.com/OPM/ResInsight/issues/12663
|
||||
|
||||
m_realizationSubSet = rangeString;
|
||||
|
||||
sendFileSetChangedSignal();
|
||||
}
|
||||
|
||||
Reference in new issue
Block a user