Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Interval array #335

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

Alex-PLACET
Copy link
Collaborator

No description provided.

@Alex-PLACET Alex-PLACET self-assigned this Jan 31, 2025
Copy link

codecov bot commented Jan 31, 2025

Codecov Report

Attention: Patch coverage is 27.56410% with 113 lines in your changes missing coverage. Please review.

Project coverage is 90.27%. Comparing base (1f003f1) to head (5d0872b).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
include/sparrow/utils/ranges.hpp 0.00% 17 Missing ⚠️
...dth_binary_layout/fixed_width_binary_reference.hpp 0.00% 15 Missing ⚠️
...e_binary_layout/variable_size_binary_reference.hpp 0.00% 15 Missing ⚠️
...de/sparrow/layout/temporal/timestamp_reference.hpp 0.00% 10 Missing ⚠️
include/sparrow/utils/nullable.hpp 0.00% 8 Missing ⚠️
include/sparrow/array.hpp 0.00% 4 Missing ⚠️
include/sparrow/layout/array_base.hpp 0.00% 4 Missing ⚠️
...nclude/sparrow/layout/dictionary_encoded_array.hpp 0.00% 4 Missing ⚠️
include/sparrow/layout/list_layout/list_value.hpp 0.00% 4 Missing ⚠️
include/sparrow/layout/null_array.hpp 0.00% 4 Missing ⚠️
... and 8 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #335      +/-   ##
==========================================
- Coverage   92.02%   90.27%   -1.75%     
==========================================
  Files          83       84       +1     
  Lines        6242     6460     +218     
==========================================
+ Hits         5744     5832      +88     
- Misses        498      628     +130     
Flag Coverage Δ
unittests 90.27% <27.56%> (-1.75%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant