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.

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

Make Epochs a DynamicTable #664

Closed
4 tasks done
ajtritt opened this issue Oct 10, 2018 · 2 comments
Closed
4 tasks done

Make Epochs a DynamicTable #664

ajtritt opened this issue Oct 10, 2018 · 2 comments
Assignees
Milestone

Comments

@ajtritt
Copy link
Member

ajtritt commented Oct 10, 2018

Problem/Use Case

The epoch table is currently a compound-dtype table. This makes it hard to extend and inefficient for querying along a single feature.

Part of this move of making Epochs a DynamicTable should also involve removing the use of RegionReferences for indexing into an array of TimeSeriesIndex. Removing this will make the schema less dependent on HDF5 primitives and make it easier to use alternative backends.

Relates to #643

Checklist

  • Have you ensured the feature or change was not already reported ?
  • Have you included a brief and descriptive title?
  • Have you included a clear description of the problem you are trying to solve?
  • Have you included a minimal code snippet that reproduces the issue you are encountering?
@bendichter
Copy link
Contributor

@ajtritt close via #690?

@ajtritt
Copy link
Member Author

ajtritt commented Nov 2, 2018

oops. close via #682

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

No branches or pull requests

2 participants