GEOS-DEV / GEOS

GEOS Simulation Framework
GNU Lesser General Public License v2.1
203 stars 80 forks source link

Feature/byer3/thermal well #3156

Open tjb-ltk opened 1 month ago

tjb-ltk commented 1 month ago

Thermal compositional well dev

1) code still has some debug statements to remove 2) test mdoels still need to be submitted

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 46.08509% with 1508 lines in your changes missing coverage. Please review.

Project coverage is 55.61%. Comparing base (d3783ad) to head (b3ed475). Report is 34 commits behind head on develop.

:exclamation: Current head b3ed475 differs from pull request most recent head 371eb04

Please upload reports for the commit 371eb04 to get more accurate results.

Files Patch % Lines
...ells/ThermalCompositionalMultiphaseWellKernels.hpp 0.00% 290 Missing :warning:
...Solvers/fluidFlow/wells/PerforationFluxKernels.hpp 0.00% 253 Missing :warning:
...dFlow/wells/CompositionalMultiphaseWellKernels.hpp 38.79% 183 Missing :warning:
...rs/multiphysics/CoupledReservoirAndWellKernels.hpp 0.00% 122 Missing :warning:
...onents/fieldSpecification/SourceFluxStatistics.cpp 33.11% 103 Missing :warning:
...rc/coreComponents/common/KernelLaunchSelectors.hpp 11.25% 71 Missing :warning:
...physicsSolvers/fluidFlow/wells/SinglePhaseWell.cpp 72.06% 50 Missing :warning:
...cs/SinglePhasePoromechanicsConformingFractures.cpp 0.00% 37 Missing :warning:
...dFlow/wells/CompositionalMultiphaseWellKernels.cpp 49.29% 36 Missing :warning:
...ies/ThermalCompositionalMultiphaseReservoirFVM.hpp 0.00% 34 Missing :warning:
... and 47 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #3156 +/- ## =========================================== + Coverage 53.57% 55.61% +2.04% =========================================== Files 1003 1033 +30 Lines 85297 88154 +2857 =========================================== + Hits 45695 49030 +3335 + Misses 39602 39124 -478 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

paveltomin commented 2 weeks ago

@tjb-ltk is it ready to be cleaned up, re-reviewed and merged?

tjb-ltk commented 2 weeks ago

No.. in training ... probably by Tuesday