Skip to content

refactor: create enums for strings in dataclasses#704

Merged
JoanFM merged 6 commits into
mainfrom
refactor-449-const-metadata-keys
Nov 2, 2022
Merged

refactor: create enums for strings in dataclasses#704
JoanFM merged 6 commits into
mainfrom
refactor-449-const-metadata-keys

Conversation

@Jackmin801

Copy link
Copy Markdown
Contributor

@codecov

codecov Bot commented Oct 28, 2022

Copy link
Copy Markdown

Codecov Report

Merging #704 (1a3f0d6) into main (3f07f52) will increase coverage by 2.59%.
The diff coverage is 97.72%.

@@            Coverage Diff             @@
##             main     #704      +/-   ##
==========================================
+ Coverage   85.95%   88.55%   +2.59%     
==========================================
  Files         133      134       +1     
  Lines        6538     6555      +17     
==========================================
+ Hits         5620     5805     +185     
+ Misses        918      750     -168     
Flag Coverage Δ
docarray 88.55% <97.72%> (+2.59%) ⬆️

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

Impacted Files Coverage Δ
docarray/dataclasses/setter.py 94.00% <88.88%> (+0.12%) ⬆️
docarray/dataclasses/enums.py 100.00% <100.00%> (ø)
docarray/dataclasses/getter.py 100.00% <100.00%> (ø)
docarray/dataclasses/types.py 93.51% <100.00%> (-0.35%) ⬇️
docarray/document/mixins/multimodal.py 93.96% <100.00%> (ø)
docarray/array/storage/weaviate/find.py 88.00% <0.00%> (+1.33%) ⬆️
docarray/array/storage/sqlite/getsetdel.py 100.00% <0.00%> (+2.38%) ⬆️
docarray/array/storage/annlite/getsetdel.py 100.00% <0.00%> (+2.63%) ⬆️
docarray/array/storage/elastic/getsetdel.py 100.00% <0.00%> (+3.38%) ⬆️
docarray/array/storage/weaviate/getsetdel.py 100.00% <0.00%> (+5.26%) ⬆️
... and 8 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Jackmin801 Jackmin801 force-pushed the refactor-449-const-metadata-keys branch from 264e73f to dd2ee41 Compare October 28, 2022 13:52
@Jackmin801 Jackmin801 force-pushed the refactor-449-const-metadata-keys branch from dd2ee41 to ab0bbf2 Compare October 28, 2022 13:54
Comment thread docarray/dataclasses/getter.py Outdated
@Jackmin801 Jackmin801 force-pushed the refactor-449-const-metadata-keys branch from ab0bbf2 to 8a1038d Compare October 28, 2022 14:16
@github-actions github-actions Bot added size/m and removed size/l labels Oct 28, 2022
@Jackmin801 Jackmin801 force-pushed the refactor-449-const-metadata-keys branch from 9c4fedd to aa8b2f3 Compare October 28, 2022 14:26
@Jackmin801 Jackmin801 force-pushed the refactor-449-const-metadata-keys branch from aa8b2f3 to b853393 Compare October 28, 2022 14:26
Comment thread docarray/dataclasses/types.py
@Jackmin801 Jackmin801 marked this pull request as ready for review November 1, 2022 13:17
@JoanFM JoanFM merged commit 6b152e1 into main Nov 2, 2022
@JoanFM JoanFM deleted the refactor-449-const-metadata-keys branch November 2, 2022 07:58
@JoanFM JoanFM linked an issue Nov 4, 2022 that may be closed by this pull request
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.

Refactor: collect possible keys of Document _metadata and store globally

4 participants