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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade openai version in documentation notebooks #1903

Merged
merged 12 commits into from May 13, 2024
Merged

Conversation

kevinmessiaen
Copy link
Member

Description

Upgrade openai version in documentation notebooks since we don't support openai<1 anymore

Related Issue

Type of Change

  • 馃摎 Examples / docs / tutorials / dependencies update
  • 馃敡 Bug fix (non-breaking change which fixes an issue)
  • 馃 Improvement (non-breaking change which improves an existing feature)
  • 馃殌 New feature (non-breaking change which adds functionality)
  • 馃挜 Breaking change (fix or feature that would cause existing functionality to change)
  • 馃攼 Security fix

@kevinmessiaen kevinmessiaen marked this pull request as ready for review April 23, 2024 08:02
@kevinmessiaen kevinmessiaen linked an issue May 2, 2024 that may be closed by this pull request
],
"outputs": [
{
"ename": "ModuleNotFoundError",
Copy link
Member

Choose a reason for hiding this comment

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

Some of the notebook outputs contain errors

# Conflicts:
#	docs/open_source/scan/scan_llm/first.md
#	docs/open_source/testset_generation/rag_evaluation/index.md
#	docs/reference/notebooks/LLM_QA_Documentation.ipynb
#	docs/reference/notebooks/LLM_QA_Winter_Olympics.ipynb
Copy link

sonarcloud bot commented May 6, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@luca-martial luca-martial merged commit 88e3cd4 into main May 13, 2024
16 checks passed
@luca-martial luca-martial deleted the doc/upgrade-openai branch May 13, 2024 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Wrapping custom application endpoint not working
2 participants