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

Issue 56: Improve reproducibility ease #57

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

athowes
Copy link

@athowes athowes commented May 20, 2024

Will close #56.

As part of figuring out how epidist works for the refactor (e.g. epinowcast/epidist#46) @seabbs and I discussed me running through the targets paper code to see how all the functions are used.

In doing this, I'll also have a go at making it easier / more visible for others to reproduce the analysis in the paper.

@athowes athowes changed the title Issue 56: improve reproducibility ease Issue 56: Improve reproducibility ease May 20, 2024
@seabbs
Copy link
Collaborator

seabbs commented May 20, 2024

Really great idea!

@athowes
Copy link
Author

athowes commented May 23, 2024

Logging things to do as I go:

  • Remove mentions of dynamicaltruncation
  • Move away from using DT style (if not suitable for package functions then not suitable for paper code?)

@seabbs
Copy link
Collaborator

seabbs commented May 24, 2024

Move away from using DT style (

I think we can keep here as it will be quite a big rewrite and its not such an issue keeping this fixed to the dev version of data.table.

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.

Could include paper source code in repo?
2 participants