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.

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

ClusterTable memory requirements #127

Closed
ms609 opened this issue Mar 25, 2022 · 0 comments · Fixed by #130
Closed

ClusterTable memory requirements #127

ms609 opened this issue Mar 25, 2022 · 0 comments · Fixed by #130

Comments

@ms609
Copy link
Owner

ms609 commented Mar 25, 2022

Running consensus_info() with 36000 trees requires a vector of 36000 ClusterTables, which requires more memory than is available.

Can we reduce the memory requirement of a ClusterTable?
(Perhaps we need to operate on the heap rather than the stack?)
@ms609 ms609 transferred this issue from ms609/TreeDist Jun 9, 2022
ms609 added a commit that referenced this issue Aug 25, 2022
@ms609
Motivation: #127
a1b7c95
@ms609 ms609 mentioned this issue Aug 25, 2022
Merged
ms609 added a commit that referenced this issue Aug 25, 2022
@ms609
Consensus of many trees (#130)
9f2e92b 
Fixes #127
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Consensus of many trees ms609/TreeTools
1 participant
@ms609