epiforecasts / ringbp

Simulate infectious disease transmission with contact tracing
http://epiforecasts.io/ringbp/
Other
15 stars 10 forks source link

Prospective refactor #77

Open joshwlambert opened 1 month ago

joshwlambert commented 1 month ago

This pull request makes a series of changes to the {ringbp} to hopefully improve the package.

This PR is not for merging. It is purely to demonstrate the types of changes that could be made in {ringbp}.

If this PR contains changes that we want to include in {ringbp} then I can make specific feature branches and tag the relevant issues. In this PR I have not related the changes to any existing issues in the {ringbp} GitHub repository as this was a more general exploration of how the package can be improved.

This is not a full refactor, but a start of to give a flavour of the improvements, without sinking too much time if we decide this is not the best direction of development for the package. Some changes include:

joshwlambert commented 3 weeks ago

Thanks for the suggestions @sbfnk. Are you happy for me to work through the {ringbp} issues and make some of the changes included in this PR along the way?

sbfnk commented 3 weeks ago

Yes, that would be great!