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

Replace Table class with a NamedTuple #92

Closed
ManuelAlvarezC opened this issue Feb 7, 2019 · 0 comments
Closed

Replace Table class with a NamedTuple #92

ManuelAlvarezC opened this issue Feb 7, 2019 · 0 comments
Labels
internal The issue doesn't change the API or functionality
Milestone
0.1.1

Comments

@ManuelAlvarezC
Copy link
Contributor

Currently, the class sdv.data_navigator.Table has no methods and serves only to keep together two attributes. This can be implemented using collections.namedtuple which will offer the same features, but are much better performance-wise.
@ManuelAlvarezC ManuelAlvarezC added the internal The issue doesn't change the API or functionality label Feb 7, 2019
ManuelAlvarezC added a commit that referenced this issue Feb 18, 2019
@ManuelAlvarezC
Merge pull request #94 from Aylr/Aylr/convert-table-to-named-tuple
a2e3f70 
* fixes #92 converted Table to a namedtuple
@csala csala added this to the 0.1.1 milestone Mar 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
internal The issue doesn't change the API or functionality
Projects
None yet
Milestone
0.1.1
Development

No branches or pull requests
2 participants
@csala @ManuelAlvarezC