-
Notifications
You must be signed in to change notification settings - Fork 89
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Make the rock toughness scalable with the fracture size and test anisotropic toughness #3310
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3310 +/- ##
===========================================
- Coverage 56.89% 56.87% -0.02%
===========================================
Files 1157 1157
Lines 101214 101244 +30
===========================================
Hits 57582 57582
- Misses 43632 43662 +30 ☔ View full report in Codecov by Sentry. |
src/coreComponents/physicsSolvers/surfaceGeneration/SurfaceGenerator.hpp
Outdated
Show resolved
Hide resolved
name="KIC_21" | ||
initialCondition="1" | ||
setNames="{ all }" | ||
objectPath="ElementRegions" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i guess the anisotropic KI will not affect the propagation direction (still perpendicular to the minimum principal stress) but the propagation speed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, correct. The fracture will remain in the same plane. It changes the propagation speed in the two directions though. It's quite useful to match field data.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to encapsulate everything related to the rock toughness into a constitutive model and completely remove the rockToughness
as a parameter of the SurfaceGenerator
. However, I am okay with merging this first and then introduce:
- 1 a PR to create an appropriate constitutive model. I think the KIC computations could be moved into a costitutive model too.
- 2 a PR to replace the current calculations with the new constitutive model.
Adding a test is good but I would take the simple example we used in the FORGE report (with the idealized stress profile) an make an advanced example in readthedocs from it. |
@frankfeifan Questions about adding a size based toughness modification:
|
@rrsettgast Please find my replies below:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should make a note to follow up on different methods for calculating the factor.
This PR is to add and test features of rock toughness.
Scalable rock toughness
a new feature that makes the rock toughness scalable with the fracture size if the hydraulic fracture is very large. The underlying mechanism is that the cohesive region of rock fractures may scale up with the fracture size, thereby increasing the critical fracture energy and thus toughness. The specific formulation is
KIC = KIC_base*(1 + scalingFactor*sqrt(L))
whereL
is the distance to the fracture origin.Here are the changes in this PR:
rockToughness
tobaseRockToughness
, and addtoughnessScalingFactor
andfractureOrigin
inSurfaceGenerator
.fractureOrigin
is manually input now, but can be removed later if we can find the fracture origin from the initial fracture geometry.K_IC
by making it a field variable infractureSubRegion
,so that one can visualize the toughness in vtk.Relevant references: Scholz, 2010, McClure, 2023
Anisotropic toughness
Existing
hydrofracture
solver has the capability to assign anisotropic toughness, but looks like there is no relevant test. An example is added to test this feature. Also, some changes may be needed to make this feature easier to use for users.