Skip to content

Enhance gravprisms -D with +c for computing contrasts#6894

Merged
PaulWessel merged 1 commit intomasterfrom
prism-dens-contrast
Jul 18, 2022
Merged

Enhance gravprisms -D with +c for computing contrasts#6894
PaulWessel merged 1 commit intomasterfrom
prism-dens-contrast

Conversation

@PaulWessel
Copy link
Member

It occurred to me that it is convenient to allow -D to be used not only to set a constant density contrast for all prisms, but with a modifier (+c) instead compute the density contrast between the variable prism densities and the constant given in -D. Otherwise, I need to compute two sets of prisms just to be able to display the densities and to compute gravity using a contrast with sea water. This PR adds the +c modifier. It basically reverts very recent behavior implemented after 6.4 so it has no backwards complications. FYI, @joa-quim.

It occurred to me that it is convenient to allow -D to be used not only to set a constant density contrast for all prisms, but with a modifier (+c) instead compute the density contrast between the variable prism densities and the constant given in -D.  Otherwise I may need to compute two sets of prisms just to be able to display the densities and to compute gravity using a contrast with sea water.  This PR adds the +c modifier.  It basically reverts very recent behavior implemented after 6.4 so it has no release effect.
@PaulWessel PaulWessel added the enhancement Improving an existing feature label Jul 17, 2022
@PaulWessel PaulWessel added this to the 6.5.0 milestone Jul 17, 2022
@PaulWessel PaulWessel self-assigned this Jul 17, 2022
@PaulWessel PaulWessel merged commit 89620db into master Jul 18, 2022
@PaulWessel PaulWessel deleted the prism-dens-contrast branch July 18, 2022 08:35
@PaulWessel
Copy link
Member Author

Note FYI: I also added a modifier +rrhor to -D. The reason is as follows:

I have a variable density grid used with topography to drive flexure. This is done via -Hrho.grd. However, the density structure rho(x,y) is not perfectly known and I will be running numerical experiments to modify the rho.grd grid by a small scalar 1+f (with f in the ±0.2 range and determine the f that minimizes some rms). So, multiplying rho(x,y) by f changes the mean load density. When the infill density and the load density are set separately then we do the Wessel [2001;2016] approximation which requires these two densities, and now the rho load density will change. However, I wish to keep the same root density (and infill density) in all these experiments, hence I wish to pass that constant root density and the scaled rho.grd so that the approximation stays the same by just adjusting the topography by 1+f. That way the flexural response to me changing 1+f is given by (1+f) times w(x,y).

Anyway, I suspect I am the only one doing this, and it does not affect any tests.

@maxrjones maxrjones added the add-changelog Add PR to the changelog label Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add-changelog Add PR to the changelog enhancement Improving an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants