Commit Graph

26 Commits

Author SHA1 Message Date
Joakim Hove
cf61068ad1 Merge pull request #587 from bska/compare-ecl-ignoreextra
Make Summary Comparison Able to Ignore Different Number of Keywords
2018-12-13 21:31:28 +01:00
Bård Skaflestad
9098316775 Make Summary Comparison Able to Ignore Different Number of Keywords
This commit adds the ability of SummaryRegressionTest to bypass the
check for equal number keywords/summary vectors in the two result
sets being compared.  The primary purpose of this is to continue
comparing contents of existing summary vectors if new summary
keywords are introduced in a GitHub Pull Request.

Extend the 'compareECL' utility to take advantage of this ability.
2018-12-13 20:10:34 +01:00
Joakim Hove
1249ef0dd2 Removed unused file compareSummary.cpp 2018-12-13 19:16:05 +01:00
Atgeirr Flø Rasmussen
71b0c2883f Avoid shadowing of variables. 2018-12-10 16:10:25 +01:00
Joakim Hove
f414f5b8ad Adapt to C++ changes in libecl 2018-11-26 17:30:09 +01:00
Joakim Hove
7cd29ea7d7 Add option to ignore extra keywords in the regresstion testing 2018-09-11 13:34:51 +02:00
Arne Morten Kvarving
cb525c5c75 changed: fold compareSummary into compareECL application 2018-09-10 08:42:26 +02:00
Arne Morten Kvarving
b877c8b340 changed: put Deviation struct in a separate header
ODR
2018-09-07 15:33:50 +02:00
Arne Morten Kvarving
7a034ffd01 changed: split ECL(Integration|Regression)Test classes to separate files
to unify code with the summary comparison classes
2018-09-07 15:33:46 +02:00
Arne Morten Kvarving
6e0c31bed4 changed: make getCellVolume a static member of ECLFilesComparator
needed in upcoming splitting commit
2018-09-07 14:38:03 +02:00
Arne Morten Kvarving
ef63206925 changed: rename (Integration|Regression)Test ECL(Integration|RegressionTest)
for clarity
2018-09-07 14:38:03 +02:00
Arne Morten Kvarving
0b6d36658d changed: pass basenames as std::string
to unify code with the other comparison classes
2018-09-07 14:38:03 +02:00
Arne Morten Kvarving
44f3b86547 changed: rename the summary regression/integrationtest classes
it restores c++ ODR which is nice, but the main reason is to
allow usage of summary and generic comparison classes in the
the same application.
2018-09-07 14:38:03 +02:00
Arne Morten Kvarving
bee4590aaa changed: move the comparison classes out of libopmcommon
these classes are really not made for reusability.
thus they should only be built into the applications.
2018-09-07 14:38:03 +02:00
Arne Morten Kvarving
e072365234 added: error analysis support in compareSummary / compareECL
useful for debugging purposes. replaces scripts in opm-simulators
2018-08-14 11:02:56 +02:00
Atgeirr Flø Rasmussen
cfb3d34467 Add more flexibility to compareECL.
In particular:
 - Grid comparison will now also check that the active cells are the same,
   not just that the number of active cells is identical.
 - In the grid comparison, checking volumes is now optional, from compareECL
   this feature is used by passing the '-V' option on the command line.
2018-06-27 15:22:55 +02:00
Markus Blatt
d50ccf507d Fixes some problems with option parsing.
Namely, that m requires an argument and not n as indicated to
getopt. This caused the help message to be printed whenever users
did not want exceptions to be thrown.

Additionally we fix the error message for options that need an argument
but do not get one.
2017-04-19 15:14:25 +02:00
Joakim Hove
01f43dc6ca Change in ert data_type for ecl_kw. 2017-03-25 21:23:07 +01:00
Arne Morten Kvarving
594572152a added: option to not throw on error in restart/init regression test
useful for doing a more thorough analysis of the failure
2017-02-16 15:51:11 +01:00
Arne Morten Kvarving
97a7ca3013 added: option to not throw on error in summary regression test
useful for doing a more thorough analysis of the failure
2017-02-03 09:40:54 +01:00
Atgeirr Flø Rasmussen
a9f5fcdfd5 Silence various warnings.
Mostly shadowing, also unused variable and unused argument.
2016-09-23 15:02:36 +02:00
Arne Morten Kvarving
a403a8d21c Changed to two applications for comparison
- restartRegressionTest, initRegressionTest was squashed to one
 application, which also supports integration test as well as test
 of restart files (non-unified) and .RFT files.

- summaryRegressionTest was renamed compareSummary -- now it can
 execute an integration test as well as regression test. Also other
 improvements were made.
2016-09-19 14:36:03 +02:00
babrodtk
ec43c9c86f Fixed warnings 2016-09-06 07:54:21 +02:00
Petter Taule
163f5a5cf5 Added applications using the regression test libraries
Added restartRegressionTest.cpp, initRegressionTest.cpp, rftRegressionTest.cpp, summaryRegressionTest.cpp, and modifying CMakeLists_files.cmake to include these.
2016-07-22 09:19:46 +02:00
jokva
6ef249b9a7 Revert "C++ regression test of eclipse files (initial, unified restart and summary)" 2016-07-21 16:37:18 +02:00
Petter Taule
2f69ec7e23 Added applications using the regression test libraries
Added restartRegressionTest.cpp, initRegressionTest.cpp, rftRegressionTest.cpp, summaryRegressionTest.cpp, and modifying CMakeLists_files.cmake to include these.
2016-07-21 16:02:41 +02:00