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

Issue/95/index files #99

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Issue/95/index files #99

wants to merge 17 commits into from

Conversation

eacharles
Copy link
Collaborator

Problem & Solution Description (including issue #)

Code Quality

  • My code follows the code style of this project
  • I have written unit tests or justified all instances of #pragma: no cover; in the case of a bugfix, a new test that breaks as a result of the bug has been added
  • My code contains relevant comments and necessary documentation for future maintainers; the change is reflected in applicable demos/tutorials (with output cleared!) and added/updated docstrings use the NumPy docstring format
  • Any breaking changes, described above, are accompanied by backwards compatibility and deprecation warnings

@eacharles eacharles mentioned this pull request May 29, 2024
Open
sidneymau and others added 15 commits September 3, 2024 13:20
@sidneymau @eacharles
Initial commit for pyarrow support (#93)
0f66efd 
* Initial commit for pyarrow support

* fix up some unit test, not full coverage yet

* Cast arrow array to numpy array

* WIP, unit tests

* get unit test working

* Propagate metadata to pyarrow tables

* tweak for full coverage

---------

Co-authored-by: Eric Charles <echarles@gmail.com>
@eacharles
added mergeUtils.py
e9249a4
@eacharles
switch to using cli
af288a4
@eacharles
added sliceUtils and get unit tests working
8d70bf0
@eacharles
minor delinting
7b54a29
@eacharles
WIP for index files
0eb3e6f
@sidneymau @eacharles
Initial commit for pyarrow support (#93)
5987fc3 
* Initial commit for pyarrow support

* fix up some unit test, not full coverage yet

* Cast arrow array to numpy array

* WIP, unit tests

* get unit test working

* Propagate metadata to pyarrow tables

* tweak for full coverage

---------

Co-authored-by: Eric Charles <echarles@gmail.com>
@eacharles
WIP for index files
764dd77
@eacharles
WIP on index files
efe83c4
@eacharles
added stuff about index files to ioUtils
1a5becd
@eacharles
added sliceUtils and get unit tests working
eb8846b
@eacharles
Get unit tests working, also, move to using data_tables fixture
08ac252
@eacharles
added read for index files
9d173b6
@eacharles
fix up help message for make-index
d5e7c04
@eacharles
fix doc message in cli
f38310f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eacharles @sidneymau