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

[FEAT] Micropartition integration and tests #1502

Merged
merged 70 commits into from
Oct 25, 2023

Conversation

jaychia
Copy link
Contributor

@jaychia jaychia commented Oct 19, 2023

  1. Adds a DAFT_MICROPARTITIONS=1 flag to enable usage of MicroPartition instead of Table for execution
  2. Adds tests and implementations for micropartition methods

@github-actions github-actions bot added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 19, 2023
@codecov
Copy link

codecov bot commented Oct 19, 2023

Codecov Report

Merging #1502 (1dd87d0) into main (15085f5) will increase coverage by 0.47%.
The diff coverage is 90.40%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1502      +/-   ##
==========================================
+ Coverage   74.73%   75.21%   +0.47%     
==========================================
  Files          60       61       +1     
  Lines        6140     6335     +195     
==========================================
+ Hits         4589     4765     +176     
- Misses       1551     1570      +19     
Files Coverage Δ
daft/execution/execution_step.py 92.83% <100.00%> (+0.01%) ⬆️
daft/runners/ray_runner.py 90.55% <ø> (ø)
daft/table/__init__.py 100.00% <100.00%> (ø)
daft/table/table.py 82.06% <100.00%> (+0.12%) ⬆️
daft/viz/dataframe_display.py 93.18% <100.00%> (ø)
daft/table/micropartition.py 89.83% <89.83%> (ø)

@jaychia jaychia force-pushed the sammy/micropartition-methods branch from 0636998 to 5579f75 Compare October 19, 2023 14:26
@jaychia jaychia force-pushed the sammy/micropartition-methods branch from 0f3f6b1 to a7ed8c7 Compare October 23, 2023 16:55
@samster25 samster25 merged commit f547b86 into main Oct 25, 2023
37 checks passed
@samster25 samster25 deleted the sammy/micropartition-methods branch October 25, 2023 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants