Skip to content

feat: plot matching sprite func#265

Merged
makram93 merged 11 commits into
mainfrom
feat-plot-matching-sprite
Apr 11, 2022
Merged

feat: plot matching sprite func#265
makram93 merged 11 commits into
mainfrom
feat-plot-matching-sprite

Conversation

@makram93

@makram93 makram93 commented Apr 7, 2022

Copy link
Copy Markdown
Contributor

This PR adds functionality to show matching images in a sprite. The leftmost is the query followed by matching results. There is a gap between the query and its matches to distinguish/differentiate.
image

@github-actions github-actions Bot added the size/m label Apr 7, 2022
@makram93 makram93 changed the title feat: plot matching sprite funs feat: plot matching sprite func Apr 7, 2022
@codecov

codecov Bot commented Apr 7, 2022

Copy link
Copy Markdown

Codecov Report

Merging #265 (c88bac5) into main (134bb1e) will increase coverage by 0.89%.
The diff coverage is 69.35%.

@@            Coverage Diff             @@
##             main     #265      +/-   ##
==========================================
+ Coverage   85.16%   86.05%   +0.89%     
==========================================
  Files         134      134              
  Lines        6160     6225      +65     
==========================================
+ Hits         5246     5357     +111     
+ Misses        914      868      -46     
Flag Coverage Δ
docarray 86.05% <69.35%> (+0.89%) ⬆️

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

Impacted Files Coverage Δ
docarray/document/mixins/plot.py 77.63% <68.85%> (-5.89%) ⬇️
docarray/__init__.py 100.00% <100.00%> (ø)
docarray/array/mixins/find.py 88.88% <0.00%> (-0.16%) ⬇️
docarray/math/ndarray.py 90.00% <0.00%> (+0.34%) ⬆️
docarray/array/mixins/io/binary.py 97.45% <0.00%> (+1.27%) ⬆️
docarray/array/mixins/match.py 81.25% <0.00%> (+6.25%) ⬆️
docarray/array/mixins/io/pushpull.py 87.23% <0.00%> (+8.51%) ⬆️
docarray/helper.py 82.71% <0.00%> (+14.48%) ⬆️
docarray/array/mixins/post.py 84.37% <0.00%> (+71.87%) ⬆️

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 eaf43e2...c88bac5. Read the comment docs.

@davidbp davidbp closed this Apr 7, 2022
@davidbp davidbp reopened this Apr 7, 2022
Comment thread docarray/document/mixins/plot.py Outdated
Comment thread docarray/document/mixins/plot.py Outdated
Comment thread docarray/document/mixins/plot.py Outdated

@bwanglzu bwanglzu left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left some comments

Comment thread docarray/document/mixins/plot.py Outdated
Comment thread docarray/document/mixins/plot.py Outdated
Comment thread docarray/document/mixins/plot.py Outdated
@bwanglzu

bwanglzu commented Apr 7, 2022

Copy link
Copy Markdown

@makram93 mybad, literal only works on py3.8+, please ignore my first 2 comments.

Comment thread docarray/document/mixins/plot.py Outdated
@makram93 makram93 requested a review from bwanglzu April 8, 2022 11:52

@bwanglzu bwanglzu left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@hanxiao hanxiao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@makram93 makram93 requested a review from hanxiao April 11, 2022 06:50
@makram93 makram93 merged commit acc76ca into main Apr 11, 2022
@makram93 makram93 deleted the feat-plot-matching-sprite branch April 11, 2022 09:38
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.

5 participants