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

Added a function to plot sizes over time in parallel with the DE plot… #48

Merged
merged 3 commits into from
Dec 5, 2024

Conversation

Tess-LaCoil
Copy link
Collaborator

… function. Changed wording in README to be more accessible.

… function. Changed wording in README to be more accessible.
@Tess-LaCoil Tess-LaCoil requested a review from fontikar December 4, 2024 04:29
@Tess-LaCoil Tess-LaCoil requested a review from dfalster December 5, 2024 00:53
Copy link
Member

@dfalster dfalster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great to see tests included!

@Tess-LaCoil Tess-LaCoil merged commit 787d271 into master Dec 5, 2024
4 checks passed
@Tess-LaCoil Tess-LaCoil deleted the last-minute-changes branch December 5, 2024 05:02
Tess-LaCoil added a commit that referenced this pull request Dec 5, 2024
#48) (#49)

* Added a function to plot sizes over time in parallel with the DE plot function. Changed wording in README to be more accessible.

* Updated namespace, documentation.

* Removed unused args from hmde_plot_obs_est_ind function doc.
Tess-LaCoil added a commit that referenced this pull request Dec 10, 2024
* Removed ind_y_0 prior as to include it double-counted the first observation in the likelihood. Also removed y_0_obs data from the required data structures as it is no longer used. The first observation is now fit by setting y_hat = ind_y_0, and then the likelihood that compares y_hat and y_obs does the comparison of the first observed size to the estimated value.

* Standardised prior fixed input values, (0,1) for population-level, individual-level, (0,2) for global_error_sigma. Removed Delta_hat from generated quantities.

* Corrected testing to account for removing the y_0_obs data.

* Changed name of hmde_model_name function to be hmde_model_names as consistent with the file name.

* Updated vignette to have better CI printing code.

* Added a function to plot sizes over time in parallel with the DE plot… (#48) (#49)

* Added a function to plot sizes over time in parallel with the DE plot function. Changed wording in README to be more accessible.

* Updated namespace, documentation.

* Removed unused args from hmde_plot_obs_est_ind function doc.

* Updated vB vignette to be better presented.

* Tidied up code in vignettes, re-ran Canham model with RK45 for sure this time and swapped out the saved estimates.

* Wrapped up Canham vignette for paper results.

* Wrangled some well-organised plots for the paper, added cowplot package to suggests.

* Added estimation of distance between the erroneous cluster and true parameters.

* Last few things tidied up.

* Corrected code problems in vignettes.

* Swapped to 1 core for check passing purposes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants