Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Remove discontinued OpenAPITool and OpenAIFunctionCaller experiments #143

Merged

Conversation

shadeMe
Copy link
Contributor

@shadeMe shadeMe commented Nov 28, 2024

Proposed Changes:

Remove the discontinued experiments and update the readme.

How did you test it?

Locally

Checklist

@shadeMe shadeMe requested review from a team as code owners November 28, 2024 13:27
@shadeMe shadeMe requested review from dfokina and silvanocerza and removed request for a team November 28, 2024 13:27
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@shadeMe shadeMe requested review from anakin87 and removed request for silvanocerza November 28, 2024 13:27
@coveralls
Copy link

Pull Request Test Coverage Report for Build 12069823551

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 4 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.7%) to 83.132%

Files with Coverage Reduction New Missed Lines %
util/auth.py 4 37.5%
Totals Coverage Status
Change from base Build 12060155663: 0.7%
Covered Lines: 1996
Relevant Lines: 2401

💛 - Coveralls

Copy link
Member

@anakin87 anakin87 left a comment

Choose a reason for hiding this comment

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

👍

The "Sync docs with Readme" workflow is probably failing because this PR is from a fork.

@shadeMe shadeMe merged commit 61cd118 into deepset-ai:main Nov 28, 2024
9 of 10 checks passed
@shadeMe shadeMe deleted the chore/remove-discontinued-experiments branch November 28, 2024 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants