AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

Plugin for text embedding

Open ddannyc opened this issue 2 years ago • 7 comments

Background

This pull request duplicates #2804, but that pull request did not pass the tests and may not be merged.

Changes

  • autogpt/llm_utils.py#create_embedding check and handle embedding
  • autogpt/models/base_open_ai_plugin.py add embedding plugin

Documentation

Test Plan

tests/unit/test_plugins.py::test_scan_plugins_openai

PR Quality Checklist

  • [x] My pull request is atomic and focuses on a single change.
  • [x] I have thoroughly tested my changes with multiple different prompts.
  • [x] I have considered potential risks and mitigations for my changes.
  • [x] I have documented my changes clearly and comprehensively.
  • [x] I have not snuck in any "extra" small tweaks changes

ddannyc avatar Apr 28 '23 09:04 ddannyc

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 19, 2023 3:27am

vercel[bot] avatar Apr 28 '23 09:04 vercel[bot]

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

github-actions[bot] avatar Apr 28 '23 19:04 github-actions[bot]

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

github-actions[bot] avatar Apr 29 '23 05:04 github-actions[bot]

Codecov Report

Patch coverage: 50.00% and project coverage change: -0.06 :warning:

Comparison is base (5b4bcf1) 62.67% compared to head (5372625) 62.62%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3462      +/-   ##
==========================================
- Coverage   62.67%   62.62%   -0.06%     
==========================================
  Files          74       74              
  Lines        3400     3411      +11     
  Branches      495      498       +3     
==========================================
+ Hits         2131     2136       +5     
- Misses       1120     1124       +4     
- Partials      149      151       +2     
Impacted Files Coverage Δ
autogpt/llm/llm_utils.py 63.97% <25.00%> (-2.70%) :arrow_down:
autogpt/models/base_open_ai_plugin.py 100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Apr 29 '23 06:04 codecov[bot]

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

github-actions[bot] avatar May 01 '23 18:05 github-actions[bot]

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

github-actions[bot] avatar May 02 '23 06:05 github-actions[bot]

This is a mass message from the AutoGPT core team. Our apologies for the ongoing delay in processing PRs. This is because we are re-architecting the AutoGPT core!

For more details (and for infor on joining our Discord), please refer to: https://github.com/Significant-Gravitas/Auto-GPT/wiki/Architecting

p-i- avatar May 05 '23 00:05 p-i-

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

github-actions[bot] avatar May 25 '23 18:05 github-actions[bot]

Closing this as a duplicate of #2804. Still, thanks for picking it up! Turns out, it was just the linter complaining on that PR, so would have been easy to fix. Now it took a little bit more work to get it up to date with master, but still fairly easy to fix up & merge. :)

Pwuts avatar Jul 14 '23 20:07 Pwuts