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

Reviewer 1.2 #29

Open
Eabousam opened this issue Mar 2, 2024 · 1 comment
Open

Reviewer 1.2 #29

Eabousam opened this issue Mar 2, 2024 · 1 comment

Comments

@Eabousam
Copy link
Collaborator

Eabousam commented Mar 2, 2024

The authors did not include any incorporation of the uncertainty of estimates into their evaluation...they could at the least do an analysis using the confidence intervals of the estimates and assessing the coverage of their intervals compared to the “truth” data

TODO:
Run SVI and add posterior predictive to look at coverage.
This is the only place that needs to have CIs.
Supplementary figure similar to Figure 2 but look at coverage over time

@Eabousam
Copy link
Collaborator Author

Eabousam commented Apr 8, 2024

Created coverage metric, deciding on adding noise to the figure

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

No branches or pull requests

1 participant