-
Notifications
You must be signed in to change notification settings - Fork 33
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
interface for the distance to a feature's plane #453
interface for the distance to a feature's plane #453
Conversation
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.
Thanks for making this pull request! I think this will be a useful addition. To start let's get the tester working. Could you indent the code and add world_builder/objects/distance_from_surface.h
?
10e2e40
to
dc92a24
Compare
|
dc92a24
to
fb68286
Compare
@MFraters There is some tests failing, I am not sure if I missed something (tests / test_linux_and_macos_mpi ). |
It took me a bit to get to this issue. The ubuntu-latest tester images where updated and I should concider fixing them to prevent these kind of unexpected problems in the future. Can you rebase? |
fb68286
to
5d612a5
Compare
@MFraters I have just rebased to the main branch. |
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.
Great, the tester is working again. Some small comments, mostly on documentation.
The pull request looks good to me, but can you confirm that it actually does what you want it to do when building your model? Adding a picture of that here would be cool :)
d56fd18
to
e28bdd6
Compare
Codecov Report
@@ Coverage Diff @@
## main #453 +/- ##
==========================================
+ Coverage 90.65% 90.68% +0.03%
==========================================
Files 88 89 +1
Lines 6246 6271 +25
==========================================
+ Hits 5662 5687 +25
Misses 584 584
Continue to review full report at Codecov.
|
@MFraters I have addressed everything in the comment. I am going to implement the designated plugin in aspect with this PR. I am thinking of generalizing that a little bit as well. Let's say the goal is to place particles relative to the surface of a feature defined in the WB. I'll update the progress. |
e28bdd6
to
8fd0518
Compare
Hi Menno, I have updated the CHANGELOG.md. I also have a better figure than the one I sent to you. I first put a line of particles in the harzburgite layer by 5 km. The interval between particles is 0.5 km and about 9 thousand particles are entrained in the harzburgite layer (8 thousand are not plotted below the subducting plate). The interface in this PR is used in the implementation of particles shown in this following figure: |
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.
That looks great, thanks for contributing it!
I added a simple interface that could be used by ASPECT to call the distance_point_from_curved_planes function for computing the distance to a feature's plane surface.
I have also added a test in the unit tests for testing the implementation of the interface.