Nicolás Torres Domínguez

Results 22 issues of Nicolás Torres Domínguez

* **Please check if the PR fulfills these requirements** - [ ] I have read the CONTRIBUTING guidelines - [ ] A new item has been added to `NEWS.md` -...

* **Please check if the PR fulfills these requirements** - [ ] I have read the CONTRIBUTING guidelines - [ ] A new item has been added to `NEWS.md` -...

This is to address https://github.com/epiverse-trace/serofoi/pull/83#discussion_r1294343451 in a separate PR.

simdata
warning

As noted by @ben18785 in #83, the data simulation function `generate_sim_data` in `seroprevalence_data.R` adds unnecessary information to the output dataset, namely `tsur`, `test` , `antibody` and `survey_label` . This was...

refactor
fix

This PR replace the old modelling and visualisation functions for refactored versions agreed on by the development team of the package (@zmcucunuba, @ben18785, @sumalibajaj, @jpavlich, @ekamau) earlier this year. We...

The current development version (#200) needs manual specification of `init` in [`rstan::sampling()`](https://mc-stan.org/rstan/reference/stanmodel-method-sampling.html). We should set some defaults for the different models and explore the possibility of running `rstan::optimize()` to set...

* **Please check if the PR fulfills these requirements** - [ ] I have read the CONTRIBUTING guidelines - [ ] A new item has been added to `NEWS.md` -...

As pointed out by @ben18785 in #200, the way in which we were indexing the FOI was ambiguous as we were just providing a vector with the indexes (see #206):...

Dedicated unit tests for `get_foi_index()` is missing. This should be added after or alongside the solution to #206, as the function will change substantially when considering age/time structured `foi_index` across...