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

Clarify need for borel distribution functions in the package #128

Closed
jamesmbaazam opened this issue Dec 12, 2023 · 0 comments · Fixed by #234
Closed

Clarify need for borel distribution functions in the package #128

jamesmbaazam opened this issue Dec 12, 2023 · 0 comments · Fixed by #234
Labels
documentation Improvements or additions to documentation
Milestone

Comments

@jamesmbaazam
Copy link
Member

My understanding is that these functions are not directly used in the package and are tangential in terms of scope. However, since the sampling process is derived from a branching process, it is convenient and can make sense to have them here.

If the decision is to keep them here, it would be good to call this out explicitly in the docs.

Originally posted by @Bisaloo in #122 (comment)

@jamesmbaazam jamesmbaazam added this to the v0.1.0 milestone Mar 19, 2024
@jamesmbaazam jamesmbaazam added the documentation Improvements or additions to documentation label Mar 19, 2024
@jamesmbaazam jamesmbaazam moved this from Todo to In review in epichains-v0.1.0 Mar 22, 2024
@github-project-automation github-project-automation bot moved this from In review to Done in epichains-v0.1.0 Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant