[RM/DSM] bugfix for transport porosity computation in case of DSM
The transport porosity was overwritten in case of double structure porosity model.
Todo:
-
fix bug -
update ctest -
adapt ctest
The ctest needs to be updated. But within this ctest the main features of the model are not clearly visible. Therefore I recommend to change the parameters.
Edited by steffenbeese
Merge request reports
Activity
requested review from @endJunction
assigned to @steffenbeese
mentioned in merge request !3690 (closed)
added 1 commit
- de6c9e58 - [RM/DSM] change of parameters in ctest file to see the features
added 59 commits
-
de6c9e58...4434c70d - 56 commits from branch
ogs:master
- 2694498e - [RM/DSM] bugfix for transport porosity computation in case of DSM
- 805f12de - [RM/DSM] adapt ctest result files
- bcc8d334 - [RM/DSM] change of parameters in ctest file to see the features
Toggle commit list-
de6c9e58...4434c70d - 56 commits from branch
added 1 commit
- 2eee30ff - [RM/DSM] change of parameters in ctest file to see the features
added 1 commit
- a3f2b758 - [RM/DSM] change of parameters in ctest file to see the features
added 1 commit
- f4e1b553 - [RM/DSM] change of parameters in ctest file to see the features
added 1 commit
- 520ce09e - [RM/DSM] change of parameters in ctest file to see the features
added 1 commit
- e70abe74 - [RM/DSM] change of parameters in ctest file to see the features
added 1 commit
- 834aab5e - [RM/DSM] change of parameters in ctest file to see the features
- Resolved by steffenbeese
@steffenbeese Don't worry about the other (LIE/HM) failing benchmarks. These are being fixed in other MRs. Changes look good now.
added 33 commits
-
834aab5e...0b204bc3 - 30 commits from branch
ogs:master
- cbc9f712 - [RM/DSM] bugfix for transport porosity computation in case of DSM
- 5ae2d434 - [RM/DSM] adapt ctest result files
- 4138ca0b - [RM/DSM] change of parameters in ctest file to see the features
Toggle commit list-
834aab5e...0b204bc3 - 30 commits from branch
added workflowplease review label
Please register or sign in to reply