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

Update utils.trk_stats to return a dictionary of stats as output #89

Merged
merged 4 commits into from Jan 13, 2022

Conversation

msschwartz21
Copy link
Member

The trk_stats function originally printed stats, but did not return them to the user. This PR updates the function to return a dictionary of statistics and makes it possible to input X, y and lineages instead of loading the data from filename.

Copy link
Contributor

@willgraf willgraf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@willgraf willgraf added the fix Fixes a bug label Jan 12, 2022
@msschwartz21 msschwartz21 merged commit a1a12fb into master Jan 13, 2022
@msschwartz21 msschwartz21 deleted the mrgn/trk-stats branch January 13, 2022 03:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants