Skip to content

feat(protobuf): add arg for compatible ndarray type#169

Merged
hanxiao merged 2 commits intomainfrom
feat-add-ndarray-type
Mar 4, 2022
Merged

feat(protobuf): add arg for compatible ndarray type#169
hanxiao merged 2 commits intomainfrom
feat-add-ndarray-type

Conversation

@hanxiao
Copy link
Copy Markdown
Member

@hanxiao hanxiao commented Mar 4, 2022

No description provided.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 4, 2022

Codecov Report

Merging #169 (7f50cec) into main (df59998) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #169      +/-   ##
==========================================
+ Coverage   84.16%   84.17%   +0.01%     
==========================================
  Files         115      115              
  Lines        5026     5030       +4     
==========================================
+ Hits         4230     4234       +4     
  Misses        796      796              
Flag Coverage Δ
docarray 84.17% <100.00%> (+0.01%) ⬆️

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

Impacted Files Coverage Δ
docarray/__init__.py 100.00% <100.00%> (ø)
docarray/array/mixins/io/binary.py 94.65% <100.00%> (ø)
docarray/document/mixins/protobuf.py 83.33% <100.00%> (ø)
docarray/proto/io/__init__.py 91.52% <100.00%> (ø)
docarray/proto/io/ndarray.py 94.95% <100.00%> (+0.17%) ⬆️

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 2dfa095...7f50cec. Read the comment docs.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2022

📝 Docs are deployed on https://ft-feat-add-ndarray-type--jina-docs.netlify.app 🎉

@hanxiao hanxiao merged commit a38e0fc into main Mar 4, 2022
@hanxiao hanxiao deleted the feat-add-ndarray-type branch March 4, 2022 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant