Skip to content
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

Update 08_passive_margin_variable_depth.md #716

Merged
merged 2 commits into from
Aug 20, 2024

Conversation

ljhwang
Copy link
Contributor

@ljhwang ljhwang commented Jun 1, 2024

This is the first commit to fix only typos and grammatical errors.

This will be followed by a second commit to modify the paragraph we discussed.

Copy link

github-actions bot commented Jun 1, 2024

Benchmark Main Feature Difference (99.9% CI)
Slab interpolation simple none 0.989 ± 0.004 (s=462) 0.989 ± 0.005 (s=450) -0.1% .. +0.1%
Slab interpolation curved simple none 0.992 ± 0.004 (s=468) 0.991 ± 0.004 (s=442) -0.1% .. +0.1%
Spherical slab interpolation simple none 1.161 ± 0.005 (s=384) 1.162 ± 0.006 (s=393) -0.1% .. +0.2%
Slab interpolation simple curved CMS 1.032 ± 0.004 (s=444) 1.033 ± 0.003 (s=430) -0.0% .. +0.1%
Spherical slab interpolation simple CMS 1.569 ± 0.010 (s=280) 1.570 ± 0.013 (s=296) -0.1% .. +0.3%
Spherical fault interpolation simple none 1.163 ± 0.013 (s=393) 1.163 ± 0.007 (s=383) -0.3% .. +0.1%
Cartesian min max surface 2.371 ± 0.027 (s=185) 2.371 ± 0.022 (s=197) -0.3% .. +0.4%
Spherical min max surface 7.041 ± 0.082 (s=57) 7.032 ± 0.037 (s=73) -0.7% .. +0.4%

@MFraters MFraters added the ready to merge Pull request is ready to merge. May be waiting for tests to complete or other reviews. label Aug 19, 2024
@MFraters MFraters force-pushed the patch-35 branch 2 times, most recently from 70b4ae1 to 37bca14 Compare August 19, 2024 18:41
ljhwang and others added 2 commits August 19, 2024 22:36
This is the first commit to fix only typos and grammatical errors.

This will be followed by a second commit to modify the paragraph we discussed.
@MFraters MFraters merged commit c7d6b21 into GeodynamicWorldBuilder:main Aug 20, 2024
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Pull request is ready to merge. May be waiting for tests to complete or other reviews.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants