Pytables: bug fixes, code cleanup, and much updated docs #2346
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1. added __str__ (to do __repr__)
2. added __delitem__ to support store deletion syntatic sugar
3. row removal in tables is much faster if rows are consecutive
4. added Term class, refactored Selection (this is backwards compatible)
Term is a concise way of specifying conditions for queries, e.g.
Term(dict(field = 'index', op = '>', value = '20121114'))
Term('index', '20121114')
Term('index', '>', '20121114')
Term('index', ['20121114','20121114'])
Term('index', datetime(2012,11,14))
Term('index>20121114')
added alias to the Term class; you can specify the nomial indexers (e.g. index in DataFrame, major_axis/minor_axis or alias in Panel)
this should close GH pandas-dev#1996
5. added Col class to manage the column conversions
6. added min_itemsize parameter and checks in pytables to allow setting of indexer columns minimum size
7. added indexing support via method create_table_index (requires 2.3 in PyTables)
btw now works quite well as Int64 indicies are used as opposed to the Time64Col which has a bug); includes a check on the pytables version requirement
this should close GH pandas-dev#698
8. signficantlly updated docs for pytables to reflect all changes; added docs for Table sections
9. BUG: a store would fail if appending but the a put had not been done before (see test_append)
this the result of incompatibility testing on the index_kind
10. BUG: minor change to select and remove: require a table ONLY if where is also provided (and not None)
all tests pass; tests added for new features
…); added test to catch
ghost
mentioned this pull request
|
Closing this, superceded by v4 (you can just keep pushing to the same PR or rebase on master if you don't want to create new pull requests) |
|
thanks ready to merge when u r ready On Nov 27, 2012, at 7:31 PM, Wes McKinney [email protected] wrote:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Should be completely backwards compatible
mainly bug fixes, code clean, and much updated docs
should merge cleanly
added str (to do repr)
added delitem to support store deletion syntatic sugar
row removal in tables is much faster if rows are consecutive
(remove also returns the number of rows removed if a table)
added Term class, refactored Selection (this is backwards compatible)
Term is a concise way of specifying conditions for queries, e.g.
this should close GH PyTables enhancements for selection #1996
added Col class to manage the column conversions
BUG: added min_itemsize parameter and checks in pytables to allow setting of indexer columns minimum size (current implemenation will truncate indexer columns that are too long in a subsequent append - loss of info)
added indexing support via method create_table_index (requires 2.3 in PyTables)
btw now works quite well as Int64 indicies are used as opposed to the Time64Col which has a bug); includes a check on the pytables version requirement
this should close GH Add option to create indexes in HDFStore if user is using PyTables Pro / PyTables 2.3+ #698
signficantlly updated docs for pytables to reflect all changes; added docs for Table sections
BUG: a store would fail if appending but the put had not been done before (see test_append)
this the result of incompatibility testing on the index_kind
BUG: minor change to select and remove: require a table ONLY if where is also provided (and not None)
all tests pass; tests added for new features
I have some implementation changes to make to Tables to make writing quite a bit faster
but will do in a future commit