Skip to content

Unify table iteration#57

Merged
molpopgen merged 1 commit intomainfrom
unify_table_iteration
Apr 14, 2021
Merged

Unify table iteration#57
molpopgen merged 1 commit intomainfrom
unify_table_iteration

Conversation

@molpopgen
Copy link
Member

@molpopgen molpopgen commented Apr 13, 2021

This PR builds on #52:

  • Redefine TableIterator to store either a reference or an object.
  • Implement standalone "get row" functions for each table.
  • For each table foo, add TableCollection::foo_iter. These are effectively convenience functions related to what came out of Table iteration #52

@molpopgen molpopgen marked this pull request as draft April 13, 2021 22:41
* Redefine TableIterator to work via references or objects
* For a table, foo, add TableIterator::foo_iter
@molpopgen molpopgen force-pushed the unify_table_iteration branch from a636177 to 3d79f8b Compare April 14, 2021 16:45
@molpopgen molpopgen marked this pull request as ready for review April 14, 2021 16:58
@molpopgen molpopgen merged commit 6b9631e into main Apr 14, 2021
@molpopgen molpopgen deleted the unify_table_iteration branch April 14, 2021 16:59
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 this pull request may close these issues.

1 participant