Skip to content

docs: complement and rewrite evaluation docs#662

Merged
JoanFM merged 1 commit intomainfrom
docs-improve-evaluation-documentation
Oct 21, 2022
Merged

docs: complement and rewrite evaluation docs#662
JoanFM merged 1 commit intomainfrom
docs-improve-evaluation-documentation

Conversation

@guenthermi
Copy link
Copy Markdown
Contributor

@guenthermi guenthermi commented Oct 21, 2022

Goals:

  • At the beginning of the page, I explain what evaluate() does a bit more clearly.

  • add documentation for the metric_names property

  • add documentation on how to implement a custom metric function

  • Restructure the docs so that it makes a bit more sense

  • check and update documentation, if required. See guide

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 21, 2022

Codecov Report

Merging #662 (0a52258) into main (191e671) will increase coverage by 1.22%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #662      +/-   ##
==========================================
+ Coverage   85.24%   86.46%   +1.22%     
==========================================
  Files         133      133              
  Lines        6762     6762              
==========================================
+ Hits         5764     5847      +83     
+ Misses        998      915      -83     
Flag Coverage Δ
docarray 86.46% <ø> (+1.22%) ⬆️

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

Impacted Files Coverage Δ
docarray/array/mixins/evaluation.py 80.72% <ø> (ø)
docarray/array/storage/weaviate/find.py 83.54% <0.00%> (-1.27%) ⬇️
docarray/array/mixins/parallel.py 66.66% <0.00%> (+3.03%) ⬆️
docarray/array/mixins/group.py 87.50% <0.00%> (+17.50%) ⬆️
docarray/array/mixins/embed.py 89.47% <0.00%> (+78.94%) ⬆️

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

@JoanFM JoanFM merged commit eccf771 into main Oct 21, 2022
@JoanFM JoanFM deleted the docs-improve-evaluation-documentation branch October 21, 2022 15:09
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.

Revise the documentation of the evaluation Mixin

3 participants