-
Notifications
You must be signed in to change notification settings - Fork 24
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
Generalization of Cas9LineageTracingDataSimulator #216
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #216 +/- ##
==========================================
+ Coverage 79.43% 79.49% +0.06%
==========================================
Files 89 89
Lines 7895 7916 +21
==========================================
+ Hits 6271 6293 +22
+ Misses 1624 1623 -1
☔ View full report in Codecov by Sentry. 📢 Have feedback on the report? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good! Just a few changes that I think we should implement before merging it in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking great! Some small changes requested around the docstring, argument list, and handling of state_priors
specified as a np.array
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Let's merge it in.
Makes
Cas9LineageTracingDataSimulator
more general by addingself.mutation_priors_per_character
which allows for different characters to have different mutation distributions. The API is the same, exceptmutation_priors
can now be a list of lengthsize_of_cassette
ornumber_of_cassettes * size_of_cassette
. For consistency,mutation_rate
now also excepts lists ofsize_of_cassette
ornumber_of_cassettes * size_of_cassette
instead of justnumber_of_cassettes * size_of_cassette
.