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

Add OpenAI integration #778

Merged
merged 3 commits into from Mar 1, 2024
Merged

Commits on Feb 28, 2024

  1. Add OpenAI integration

    - This simplifies instrumenting OpenAI calls
    - Reuses the literalai OpenAI instrumentation
    - Creates a new step for each OpenAI call, with the call details in the generation property
    tpatel committed Feb 28, 2024
    Configuration menu
    Copy the full SHA
    ffe69d6 View commit details
    Browse the repository at this point in the history

Commits on Feb 29, 2024

  1. add a 1ms delay to the on_message callback

    - This makes sure any children step starts after the parent message step
    - replaced `generation.settings.model` by `generation.model`
    tpatel committed Feb 29, 2024
    Configuration menu
    Copy the full SHA
    f2fa8af View commit details
    Browse the repository at this point in the history
  2. move the openai version check inside the instrumentation call

    - avoids erroring if a user isn't using `instrument_openai` and hasn't installed openai
    tpatel committed Feb 29, 2024
    Configuration menu
    Copy the full SHA
    e5bf729 View commit details
    Browse the repository at this point in the history