Skip to content
This repository was archived by the owner on Sep 9, 2026. It is now read-only.

refactor(array): unify summary fn on diff backends - #230

Merged
hanxiao merged 1 commit into
mainfrom
refactor-summary-backend
Mar 28, 2022
Merged

hanxiao merged 1 commit into
mainfrom
refactor-summary-backend

Conversation

@hanxiao

@hanxiao hanxiao commented Mar 28, 2022

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Mar 28, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #230 (5cf25be) into main (fda457b) will decrease coverage by 5.97%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #230      +/-   ##
==========================================
- Coverage   85.68%   79.71%   -5.98%     
==========================================
  Files         129      129              
  Lines        5870     5862       -8     
==========================================
- Hits         5030     4673     -357     
- Misses        840     1189     +349     
Flag Coverage Δ
docarray 79.71% <100.00%> (-5.98%) ⬇️

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

Impacted Files Coverage Δ
docarray/array/storage/annlite/backend.py 68.42% <ø> (-26.50%) ⬇️
docarray/array/storage/elastic/backend.py 88.42% <ø> (-7.46%) ⬇️
docarray/array/storage/qdrant/backend.py 88.76% <ø> (-7.94%) ⬇️
docarray/array/storage/sqlite/backend.py 78.37% <ø> (-13.73%) ⬇️
docarray/array/storage/weaviate/backend.py 79.82% <ø> (-7.25%) ⬇️
docarray/array/mixins/plot.py 64.10% <100.00%> (ø)
docarray/array/storage/base/backend.py 75.00% <100.00%> (+3.57%) ⬆️
docarray/array/mixins/embed.py 10.52% <0.00%> (-78.95%) ⬇️
docarray/array/mixins/io/binary.py 46.66% <0.00%> (-50.38%) ⬇️
docarray/array/mixins/io/dataframe.py 40.00% <0.00%> (-46.67%) ⬇️
... and 23 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fda457b...5cf25be. Read the comment docs.

@hanxiao
hanxiao merged commit 6dcd7f1 into main Mar 28, 2022
@hanxiao
hanxiao deleted the refactor-summary-backend branch March 28, 2022 10:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant