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

More cleanup #76

Closed
matsen opened this issue Oct 29, 2024 · 0 comments · Fixed by #77
Closed

More cleanup #76

matsen opened this issue Oct 29, 2024 · 0 comments · Fixed by #77

Comments

@matsen
Copy link
Contributor

matsen commented Oct 29, 2024

I don't think that we need normalize_sub_probs anymore

Also, this is somewhat out of order and we could give a plural to mask

       for nt_parent, mask, nt_rates, branch_length, nt_csps in zip(
            self.nt_parents,
            self.mask,
            self.nt_ratess,
            self._branch_lengths,
            self.nt_cspss,
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 a pull request may close this issue.

1 participant