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

Resampling MVP #1495

Open
wants to merge 33 commits into
base: master
Choose a base branch
from
Open

Resampling MVP #1495

wants to merge 33 commits into from

Conversation

alexowens90
Copy link
Collaborator

@alexowens90 alexowens90 commented Apr 12, 2024 •
edited

Closes #1010

@alexowens90 alexowens90 marked this pull request as draft April 12, 2024 13:17
@alexowens90 alexowens90 force-pushed the enhancement/1010/resample-mvp-rebase branch from 3f06cc7 to d3808d0 Compare May 9, 2024 09:57
@alexowens90 alexowens90 force-pushed the enhancement/1010/resample-mvp-rebase branch from d3808d0 to 2e67393 Compare May 9, 2024 13:01
@alexowens90
Comment changes
e752b65
@alexowens90
Revert change to lmdb_version_store_tiny_segment
03df99c
@alexowens90
Remove check that input has initial expected get calls in split_by_ro…
6c3739c 
…w_slice
@alexowens90
Use Bucket class in aggregation as well
519e56c
@alexowens90
Remove Pandas date_range timing/logging, and modified some formatting
37d9810
@alexowens90
Move all sorted aggregation stuff to own files
8ac1c3b
@alexowens90
Renaming refactor
cac141e
@alexowens90
Renaming refactor
28404a8
@alexowens90
Remove unused function
6464068
@alexowens90
Remove summing timestamps from supported aggregations
9e79d0f
@alexowens90
Started refactoring aggregation
76e0baf
@alexowens90
Added push_to_aggregator method
b4e6aa7
@alexowens90
USe push_to_aggregator in the other relevant place
39ab291
@alexowens90
Fixed test_resampling_unsupported_aggregation_type_combos
91080f6
@alexowens90
Factor out finalize_aggregator
d2369eb
@alexowens90
Presize output index column in blocks, and trim unused blocks at the end
32547a2
@alexowens90
USe constexpr where possible
868ffa0
@alexowens90
Reinstate all tests, reorder source files
108867a
@alexowens90
Comment changes
91e02f4
@alexowens90
Use ColumnDataIterator in copy_frame_data_to_buffer
de3bd8e
@alexowens90
Revert accidentally committed change to task scheduler
7f1d178
@alexowens90
Comment updates
e04b124
@alexowens90
Move profile_resample.py out of tests directory
bb3639c
@alexowens90
Resample docstring
d9c0506
@alexowens90
Fix mac build?
1d5ab87
@alexowens90 alexowens90 changed the title WIP: Resample draft Resampling MVP May 15, 2024
@alexowens90 alexowens90 self-assigned this May 15, 2024
@alexowens90 alexowens90 added the enhancement New feature or request label May 15, 2024
@alexowens90 alexowens90 marked this pull request as ready for review May 15, 2024 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@willdealtry willdealtry Awaiting requested review from willdealtry willdealtry is a code owner

@poodlewars poodlewars Awaiting requested review from poodlewars poodlewars is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@alexowens90 alexowens90

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Resampling MVP
1 participant
@alexowens90