feat: Immiscible water model

[dkachuma]

Completes the water on the immiscible water model. This defines a compositional fluid model in which the water component does not participate in the flash but instead makes up it's own 3rd water/aqueous phase which is immiscible with the other 2 phases.

The phase partition is done via ImmiscibleWaterFlash. This removes the water component for the negative two phase flash of the "hydrocarbon" phases and reconstitutes the result into 3 phases.

The water properties (density and viscosity) are by providing reference values at a reference pressure and temperature and providing a compressibility and a thermal expansion coefficient.

[codecov]

Codecov Report

Attention: Patch coverage is 98.16176% with 5 lines in your changes missing coverage. Please review.

Project coverage is 57.66%. Comparing base (6b07357) to head (f3b6ffc).
Report is 1 commits behind head on develop.

Files with missing lines	Patch %	Lines
...id/compositional/models/ImmiscibleWaterDensity.hpp	90.47%	2 Missing ⚠️
.../compositional/models/ImmiscibleWaterViscosity.hpp	87.50%	2 Missing ⚠️
...tutive/unitTests/testImmiscibleWaterProperties.cpp	99.11%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3236      +/-   ##
===========================================
+ Coverage    57.51%   57.66%   +0.14%     
===========================================
  Files         1134     1139       +5     
  Lines        97866    98134     +268     
===========================================
+ Hits         56285    56586     +301     
+ Misses       41581    41548      -33     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

---

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests

[paveltomin]

i am probably missing something
a bit confused here - how will it know what eos is to use for hc?

[dkachuma]

This is specified in the xml. The phases are "lined up" with the EOS e.g.

 <CompositionalThreePhaseFluidLohrenzBrayClark
      name="blackOil"
      phaseNames="{ oil, gas, water }"
      equationsOfState="{ pr, pr, immiscibleWater }" />

However, I think since the immiscibleWater "EOS" is implied in the element name (CompositionalThreePhaseFluidLohrenzBrayClark), there is no need to specify this here. I'm inclined to remove it.

[paveltomin]

move to merge queue?

[corbett5]

Tests look fine to me.

[paveltomin]

@rrsettgast please review

[paveltomin]

Waiting on code owner review from @CusiniM, @jhuang2601, and/or @rrsettgast.

[CusiniM]

LGTM