kidusasfaw / spatPomp

R package for statistical inference of spatiotemporal partially observed Markov processes
GNU General Public License v3.0
11 stars 9 forks source link

Userdata #17

Closed kingaa closed 1 year ago

kingaa commented 1 year ago

Since version 5.3 of pomp, it hasn't been necessary to call the functions {set,unset}_pomp_userdata in workhorses, although it does no harm. The pomp_load functions now handle this. Since these functions will go away at some point, this pull request removes them from spatPomp.

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage has no change and project coverage change: -0.04% :warning:

Comparison is base (5d93429) 89.04% compared to head (b6521d1) 89.01%.

:exclamation: Current head b6521d1 differs from pull request most recent head 0b85f4f. Consider uploading reports for the commit 0b85f4f to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #17 +/- ## ========================================== - Coverage 89.04% 89.01% -0.04% ========================================== Files 54 54 Lines 4985 4971 -14 ========================================== - Hits 4439 4425 -14 Misses 546 546 ``` | [Files Changed](https://app.codecov.io/gh/kidusasfaw/spatPomp/pull/17?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [src/dunit\_measure.c](https://app.codecov.io/gh/kidusasfaw/spatPomp/pull/17?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL2R1bml0X21lYXN1cmUuYw==) | `85.07% <ø> (-0.44%)` | :arrow_down: | | [src/eunit\_measure.c](https://app.codecov.io/gh/kidusasfaw/spatPomp/pull/17?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL2V1bml0X21lYXN1cmUuYw==) | `84.84% <ø> (-0.45%)` | :arrow_down: | | [src/fcstsampvar.c](https://app.codecov.io/gh/kidusasfaw/spatPomp/pull/17?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL2Zjc3RzYW1wdmFyLmM=) | `87.80% <ø> (-0.30%)` | :arrow_down: | | [src/init.c](https://app.codecov.io/gh/kidusasfaw/spatPomp/pull/17?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL2luaXQuYw==) | `86.66% <ø> (-1.57%)` | :arrow_down: | | [src/munit\_measure.c](https://app.codecov.io/gh/kidusasfaw/spatPomp/pull/17?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL211bml0X21lYXN1cmUuYw==) | `83.60% <ø> (-0.53%)` | :arrow_down: | | [src/runit\_measure.c](https://app.codecov.io/gh/kidusasfaw/spatPomp/pull/17?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3J1bml0X21lYXN1cmUuYw==) | `79.48% <ø> (-0.52%)` | :arrow_down: | | [src/vunit\_measure.c](https://app.codecov.io/gh/kidusasfaw/spatPomp/pull/17?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3Z1bml0X21lYXN1cmUuYw==) | `84.84% <ø> (-0.45%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

kingaa commented 1 year ago

Oh, I should also say, this pull request does contains some changes to the Actions. This is a separate matter.