zwicker-group / py-pde

Python package for solving partial differential equations using finite differences.
https://py-pde.readthedocs.io
MIT License
411 stars 54 forks source link

Tried adjusting to newer numba_mpi #314

Closed david-zwicker closed 2 years ago

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 8dd92c48eada4f4f9bc2090a2161943b0f84b92b into edfec49cb8abc99533ca7301a8e935299eb8e8b3 - view on LGTM.com

new alerts:

codecov[bot] commented 2 years ago

Codecov Report

Base: 87.64% // Head: 87.69% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (d27fd7e) compared to base (3e493ea). Patch coverage: 83.15% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #314 +/- ## ========================================== + Coverage 87.64% 87.69% +0.05% ========================================== Files 69 69 Lines 10350 10390 +40 ========================================== + Hits 9071 9112 +41 + Misses 1279 1278 -1 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `87.69% <83.15%> (+0.05%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zwicker-group#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://codecov.io/gh/zwicker-group/py-pde/pull/314?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zwicker-group) | Coverage Δ | | |---|---|---| | [pde/grids/boundaries/local.py](https://codecov.io/gh/zwicker-group/py-pde/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zwicker-group#diff-cGRlL2dyaWRzL2JvdW5kYXJpZXMvbG9jYWwucHk=) | `84.60% <63.15%> (ø)` | | | [pde/tools/mpi.py](https://codecov.io/gh/zwicker-group/py-pde/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zwicker-group#diff-cGRlL3Rvb2xzL21waS5weQ==) | `82.53% <84.90%> (+22.53%)` | :arrow_up: | | [pde/grids/base.py](https://codecov.io/gh/zwicker-group/py-pde/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zwicker-group#diff-cGRlL2dyaWRzL2Jhc2UucHk=) | `89.14% <90.00%> (+0.37%)` | :arrow_up: | | [pde/grids/\_mesh.py](https://codecov.io/gh/zwicker-group/py-pde/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zwicker-group#diff-cGRlL2dyaWRzL19tZXNoLnB5) | `94.92% <100.00%> (ø)` | | | [pde/solvers/explicit\_mpi.py](https://codecov.io/gh/zwicker-group/py-pde/pull/314/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zwicker-group#diff-cGRlL3NvbHZlcnMvZXhwbGljaXRfbXBpLnB5) | `91.54% <100.00%> (+3.87%)` | :arrow_up: | 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=zwicker-group). 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=zwicker-group)

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

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging d27fd7e9e230a46bf3de3fffcb59ce441c81c9c3 into edfec49cb8abc99533ca7301a8e935299eb8e8b3 - view on LGTM.com

new alerts: