Skip to content

Conversation

@PaulWessel
Copy link
Member

An interface grid is a surface that may cross zero. If it does, the prisms we create should change sign across the zero threshold and for negative z we create a prism from x to 0 and for positive z we go from 0 to z. If a second surface or constant is given then we have a layer and no density changes sign. Documentation has been updated to explain this as well and a few comments were added elsewhere. I also added a test on the density contrast (-D cannot be 0, but variable densities from a grid could be zero at times and then we skip making a dummy prism).

An interface grid is a surface that may cross zero.  If so, the prisms we create should change sign across the zero threshold and for negative z we create a prism from x to 0 and for positive z we go from 0 to z.  If a second surface or constant is given then we have a layer and no density changes sign.  Documentation has ben updated to explain this as well adn a few comments were added elsewhere.  I also added a test on the density contrast (-D cannot be 0, but variable density from grid could be zero at times and then we skip making a dummy prism).
@PaulWessel PaulWessel added documentation Improve documentation enhancement Improving an existing feature labels Jun 28, 2022
@PaulWessel PaulWessel added this to the 6.5.0 milestone Jun 28, 2022
@PaulWessel PaulWessel requested a review from joa-quim June 28, 2022 09:05
@PaulWessel PaulWessel self-assigned this Jun 28, 2022
@PaulWessel PaulWessel merged commit 1045ef3 into master Jun 28, 2022
@PaulWessel PaulWessel deleted the interface-prisms branch June 28, 2022 11:01
@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 documentation Improve documentation enhancement Improving an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants