Eventual-Inc / Daft

Distributed DataFrame for Python designed for the cloud, powered by Rust
https://getdaft.io
Apache License 2.0
1.79k stars 108 forks source link

[FEAT] Fixed Size Binary Type #2266

Open colin-ho opened 3 weeks ago

colin-ho commented 3 weeks ago

Implements a fixed size binary type and array.

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 66.14786% with 174 lines in your changes are missing coverage. Please review.

Project coverage is 79.06%. Comparing base (984631c) to head (1f80618).

:exclamation: Current head 1f80618 differs from pull request most recent head 99da6aa. Consider uploading reports for the commit 99da6aa to get more accurate results

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266/graphs/tree.svg?width=650&height=150&src=pr&token=J430QVFE89&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc)](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc) ```diff @@ Coverage Diff @@ ## main #2266 +/- ## ========================================== - Coverage 79.17% 79.06% -0.11% ========================================== Files 459 459 Lines 53206 53712 +506 ========================================== + Hits 42128 42470 +342 - Misses 11078 11242 +164 ``` | [Files](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc) | Coverage Δ | | |---|---|---| | [src/daft-core/src/array/from.rs](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&filepath=src%2Fdaft-core%2Fsrc%2Farray%2Ffrom.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc#diff-c3JjL2RhZnQtY29yZS9zcmMvYXJyYXkvZnJvbS5ycw==) | `84.09% <100.00%> (+1.02%)` | :arrow_up: | | [src/daft-core/src/array/from\_iter.rs](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&filepath=src%2Fdaft-core%2Fsrc%2Farray%2Ffrom_iter.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc#diff-c3JjL2RhZnQtY29yZS9zcmMvYXJyYXkvZnJvbV9pdGVyLnJz) | `100.00% <100.00%> (ø)` | | | [src/daft-core/src/array/growable/arrow\_growable.rs](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&filepath=src%2Fdaft-core%2Fsrc%2Farray%2Fgrowable%2Farrow_growable.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc#diff-c3JjL2RhZnQtY29yZS9zcmMvYXJyYXkvZ3Jvd2FibGUvYXJyb3dfZ3Jvd2FibGUucnM=) | `100.00% <ø> (ø)` | | | [src/daft-core/src/array/growable/mod.rs](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&filepath=src%2Fdaft-core%2Fsrc%2Farray%2Fgrowable%2Fmod.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc#diff-c3JjL2RhZnQtY29yZS9zcmMvYXJyYXkvZ3Jvd2FibGUvbW9kLnJz) | `100.00% <ø> (ø)` | | | [src/daft-core/src/array/ops/as\_arrow.rs](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&filepath=src%2Fdaft-core%2Fsrc%2Farray%2Fops%2Fas_arrow.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc#diff-c3JjL2RhZnQtY29yZS9zcmMvYXJyYXkvb3BzL2FzX2Fycm93LnJz) | `100.00% <ø> (ø)` | | | [src/daft-core/src/array/ops/get.rs](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&filepath=src%2Fdaft-core%2Fsrc%2Farray%2Fops%2Fget.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc#diff-c3JjL2RhZnQtY29yZS9zcmMvYXJyYXkvb3BzL2dldC5ycw==) | `81.50% <ø> (ø)` | | | [src/daft-core/src/array/ops/groups.rs](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&filepath=src%2Fdaft-core%2Fsrc%2Farray%2Fops%2Fgroups.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc#diff-c3JjL2RhZnQtY29yZS9zcmMvYXJyYXkvb3BzL2dyb3Vwcy5ycw==) | `98.03% <100.00%> (+0.12%)` | :arrow_up: | | [src/daft-core/src/array/ops/is\_in.rs](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&filepath=src%2Fdaft-core%2Fsrc%2Farray%2Fops%2Fis_in.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc#diff-c3JjL2RhZnQtY29yZS9zcmMvYXJyYXkvb3BzL2lzX2luLnJz) | `100.00% <ø> (ø)` | | | [src/daft-core/src/array/ops/repr.rs](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&filepath=src%2Fdaft-core%2Fsrc%2Farray%2Fops%2Frepr.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc#diff-c3JjL2RhZnQtY29yZS9zcmMvYXJyYXkvb3BzL3JlcHIucnM=) | `68.98% <100.00%> (+0.88%)` | :arrow_up: | | [src/daft-core/src/array/ops/take.rs](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree&filepath=src%2Fdaft-core%2Fsrc%2Farray%2Fops%2Ftake.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc#diff-c3JjL2RhZnQtY29yZS9zcmMvYXJyYXkvb3BzL3Rha2UucnM=) | `99.44% <100.00%> (+0.08%)` | :arrow_up: | | ... and [21 more](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc) | | ... and [4 files with indirect coverage changes](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/2266/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc)