Closed nairr closed 6 years ago
I've made the requested changes with the latest commit
OK - then you are just missing some basic tests of this class, and it is ready to be merged.
The tests should be in now. Thanks for helping me out through this!
jenkins build this with downstreams please
OK - some minor C++ comments on the test; then this is ready to be merged. But I must admit I really don't like the approach with multiple aquifer connections in one struct. Before I merge - could you point me to the downstream code using this - that the current approach is more convenient there will be the main argument to merge the current approach.
Hi @joakim-hove , sure. It's located here https://github.com/kel85uk/opm-simulators/blob/task/%235/Make_correct_initialization_procedure/opm/autodiff/BlackoilAquiferModel_impl.hpp at the init function
cleaned up the test
jenkins build this with downstreams please
An Aquancon object which: