docs: Update CO2 injection example#3661
Merged
paveltomin merged 5 commits intodevelopfrom May 28, 2025
Merged
Conversation
tjb-ltk
approved these changes
May 13, 2025
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #3661 +/- ##
===========================================
- Coverage 57.57% 57.43% -0.15%
===========================================
Files 1235 1234 -1
Lines 107102 106983 -119
===========================================
- Hits 61669 61444 -225
- Misses 45433 45539 +106 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
CusiniM
approved these changes
May 14, 2025
paveltomin
approved these changes
May 15, 2025
jhuang2601
approved these changes
May 23, 2025
Contributor
jhuang2601
left a comment
There was a problem hiding this comment.
@dkachuma Thanks for updating this case.
@margauxraguenel Please take a look on this fix.
margauxraguenel
approved these changes
May 28, 2025
margauxraguenel
left a comment
There was a problem hiding this comment.
Tested on our side, perfect!
rrsettgast
approved these changes
May 28, 2025
danielemoretto44
pushed a commit
that referenced
this pull request
Sep 22, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR updates the CO2 injection example to address the issues raised in #3660
The pvt files in this case were updated at #2933. However, the update to the
co2flash.txtfile was missed. I have therefore updated the example so that the actual files are referenced directly without having to duplicate them in the rst.The update at #2933 changed the ranges of pressure and temperature used to generate the PVT tables. These new values are more correct because they include the surface conditions required by the well. I cannot go back to the old ranges because now geos has a check to ensure that the well operates within the range of the PVT data.
According to the change at #2933, and also at #1065, I suspect the original images were generated with a temperature range that was too low. This led to the gas density being higher and therefore the gas not being as buoyant. When this was corrected, the gas became more buoyant as can be seen with the new results.
I have edited the case to make the images look a bit more like the original. I have halved the vertical permeability to curtail the gas migration upwards and reduced the porosity to 0.05 to ensure that the gas reaches the surface. The updated example has the new updated images.