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.

Sign up for GitHub

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

Jump to bottom

Change tsk_tree_t.num_samples to tsk_size_t #1723

Closed

jeromekelleher opened this issue Sep 18, 2021 · 0 comments · Fixed by #1727

Labels

C API

Milestone

Member

jeromekelleher commented Sep 18, 2021

Currently num_samples and num_tracked samples are tsk_id_t, which seems wrong. Change to tsk_size_t unless there's a good reason not to.

jeromekelleher added the C API label

jeromekelleher added this to the C API 1.0.0 milestone

benjeffery mentioned this issue

Change num_samples and num_tracked_samples to tsk_size_t #1727

Merged

mergify bot closed this as completed in #1727

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment