CO2Leakage: Improve containment plots #1226
Merged
Conversation
This file contains 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
AudunSektnanNR
force-pushed
the
co2_containment_cont_new_5
branch
from
34e4000
to
e259e92
Compare
|
Some mysterious pytest errors in: test_structural_uncertainty.py::test_default_configuration, Tried updating/rebasing to the latest version in master-branch. |
AudunSektnanNR
force-pushed
the
co2_containment_cont_new_5
branch
from
db295d1
to
b5682fb
Compare
The script generating CO2 containment data was updated and this commit ensures that the new format is read and presented properly. This includes: - Hide the plots if the data fails to load or is missing - CO2 amounts can be scaled to Mtons/fraction - Updated default 'map_attributes_names' to be consistent with recent changes in xtgeoapp_grd3dmaps - Parsing and visualization of boundary polygon has been improved - New "UNSMRY" plot aimed to show unified summary data from Pflotran or Eclipse - Support for visualizing multi polygon boundary files Other improvements: - Added option of hazardous polygon. This creates a new category "hazardous" in the containment plots (colored red). Changed color of "outside" from red to blue and color of "inside" from blue to green. - Removed mobile gas plot - Added a plot showing the contained co2 for a particular realization - Add containment plots to UNSMRY-graph, plus numerical values for differences - Add two volume types to options in Graph Settings: Source. This will use volume data (instead of mass), if available. - Change how fraction/normalization is used as unit, divide by the maximum value across all realizations, instead of scaling each realization independently. - Various minor visual changes, plot titles etc Remove some print.
AudunSektnanNR
force-pushed
the
co2_containment_cont_new_5
branch
from
b5682fb
to
d9f8047
Compare
HansKallekleiv
approved these changes
Sep 14, 2023
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.
The script generating CO2 containment data was updated and this commit ensures that the new format is read and presented properly. This includes:
Other improvements:
Note: This PR includes all changes made in the unmerged PR #1187 , as well as changes made after this.