search

SPECFEM / specfem3d

SPECFEM3D_Cartesian simulates acoustic (fluid), elastic (solid), coupled acoustic/elastic, poroelastic or seismic wave propagation in any type of conforming mesh of hexahedra (structured or not).
GNU General Public License v3.0
390 stars 223 forks source link

updates tomography file reading, FK coupling and parallel decomposer #1593

Closed danielpeter closed 1 year ago

buildbot-princeton commented 1 year ago

The files changed do not require a buildbot test. The maintainers can commit these changes.

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 19.08% and project coverage change: -1.81 :warning:

Comparison is base (c3783a9) 45.21% compared to head (b1c5aeb) 43.41%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## devel #1593 +/- ## ========================================== - Coverage 45.21% 43.41% -1.81% ========================================== Files 250 250 Lines 56474 56795 +321 ========================================== - Hits 25535 24655 -880 - Misses 30939 32140 +1201 ``` | [Impacted Files](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM) | Coverage Δ | | |---|---|---| | [src/decompose\_mesh/module\_mesh.f90](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM#diff-c3JjL2RlY29tcG9zZV9tZXNoL21vZHVsZV9tZXNoLmY5MA==) | `0.00% <0.00%> (-68.53%)` | :arrow_down: | | [src/decompose\_mesh/module\_partition.f90](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM#diff-c3JjL2RlY29tcG9zZV9tZXNoL21vZHVsZV9wYXJ0aXRpb24uZjkw) | `0.00% <0.00%> (-80.96%)` | :arrow_down: | | [src/gpu/specfem3D\_gpu\_cuda\_method\_stubs.c](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM#diff-c3JjL2dwdS9zcGVjZmVtM0RfZ3B1X2N1ZGFfbWV0aG9kX3N0dWJzLmM=) | `0.00% <0.00%> (ø)` | | | [src/specfem3D/compute\_stacey\_viscoelastic.F90](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM#diff-c3JjL3NwZWNmZW0zRC9jb21wdXRlX3N0YWNleV92aXNjb2VsYXN0aWMuRjkw) | `29.74% <0.00%> (-29.14%)` | :arrow_down: | | [src/specfem3D/couple\_with\_injection.f90](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM#diff-c3JjL3NwZWNmZW0zRC9jb3VwbGVfd2l0aF9pbmplY3Rpb24uZjkw) | `1.70% <0.00%> (-77.82%)` | :arrow_down: | | [src/specfem3D/compute\_stacey\_acoustic.f90](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM#diff-c3JjL3NwZWNmZW0zRC9jb21wdXRlX3N0YWNleV9hY291c3RpYy5mOTA=) | `32.41% <4.28%> (-26.80%)` | :arrow_down: | | [src/specfem3D/prepare\_timerun.F90](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM#diff-c3JjL3NwZWNmZW0zRC9wcmVwYXJlX3RpbWVydW4uRjkw) | `76.86% <14.28%> (-7.64%)` | :arrow_down: | | [src/specfem3D/compute\_add\_sources\_acoustic.f90](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM#diff-c3JjL3NwZWNmZW0zRC9jb21wdXRlX2FkZF9zb3VyY2VzX2Fjb3VzdGljLmY5MA==) | `52.02% <21.42%> (-1.08%)` | :arrow_down: | | [src/specfem3D/compute\_add\_sources\_viscoelastic.F90](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM#diff-c3JjL3NwZWNmZW0zRC9jb21wdXRlX2FkZF9zb3VyY2VzX3Zpc2NvZWxhc3RpYy5GOTA=) | `40.11% <50.00%> (-2.83%)` | :arrow_down: | | [src/specfem3D/compute\_add\_sources\_poroelastic.f90](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM#diff-c3JjL3NwZWNmZW0zRC9jb21wdXRlX2FkZF9zb3VyY2VzX3Bvcm9lbGFzdGljLmY5MA==) | `35.77% <75.00%> (+0.26%)` | :arrow_up: | | ... and [7 more](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM) | | ... and [79 files with indirect coverage changes](https://codecov.io/gh/SPECFEM/specfem3d/pull/1593/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM) Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SPECFEM)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

danielpeter commented 1 year ago

accept

On 13 Apr 2023, at 16:51, buildbot-princeton @.**@.>> wrote:

The files changed do not require a buildbot test. The maintainers can commit these changes.

— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/SPECFEM/specfem3d/pull/1593*issuecomment-1507111023__;Iw!!Nmw4Hv0!yV7M-XjMgBTse21MXPMDnZMKedtWPjfBSUJlUqNZ7NZwXz7S2U2EZBRGeX8WB0MMSvzcAwrAH5AeeAxOZG7qwHs0_9sW44Y$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ABMLXWNZYRQQPK4YVUD4Y7TXBAHFLANCNFSM6AAAAAAW5GHB3Q__;!!Nmw4Hv0!yV7M-XjMgBTse21MXPMDnZMKedtWPjfBSUJlUqNZ7NZwXz7S2U2EZBRGeX8WB0MMSvzcAwrAH5AeeAxOZG7qwHs0VZZS2kA$. You are receiving this because you authored the thread.Message ID: @.***>

danielpeter commented 1 year ago

new buildbot feature? let's try again manually...

buildbot-princeton commented 1 year ago

The files changed do not require a buildbot test. The maintainers can commit these changes.

buildbot-princeton commented 1 year ago

You pushed to your local branch while your previous changes are waiting to be committed. We assume that you did not intend for these changes to be part of the currently opened pull-request and ignore those changes. If you want these changes to be committed, please open a new pull-request.

Note to maintainers: when you accept the original pull-request, the message from buildbot will say that it closed the pull-request, as opposed to the usual message that it merged the pull-request. In spite of this, the initial pull-request will be merged after you accept it.