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

Opik integration #1

Open
wants to merge 148 commits into
base: main
Choose a base branch
from
Open

Opik integration #1

wants to merge 148 commits into from

Conversation

jverre
Copy link
Collaborator

@jverre jverre commented Sep 6, 2024

Description

This PR adds a new Opik integration. Opik is an open-source LLM evaluation platform built by the Comet team.

Fixes # (issue)

New Package?

Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

  • Yes
  • No

Version Bump?

Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

  • Yes
  • No

Type of Change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Added new unit/integration tests
  • Added new notebook (that tests end-to-end)
  • I stared at the code and made sure it makes sense

The integration was checked by running:

  1. The example script provided in llama-index-integrations/callbacks/llama-index-callbacks-opik/examples/opik_example.py
  2. Running the example cookbook provided in https://www.comet.com/docs/opik/cookbook/llama-index

Suggested Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added Google Colab support for the newly added notebooks.
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I ran make format; make lint to appease the lint gods

nightosong and others added 30 commits August 27, 2024 18:25
* add xinference rerank class

* Update base.py

use def to replace lamabda

* Update base.py

rm useless init

* Update base.py

fix linting

---------

Co-authored-by: Logan <[email protected]>
* add optimum intel with ipex backend to llama-index-integration with notebook example

* fix typo

* bumped dependency versions and upgraded to newer llama-index-utils-huggingface package

* add gaudi-tei link in example notebook

---------

Co-authored-by: Rita Brugarolas <[email protected]>
Documenting: Default workflows for nested workflows, streaming events, unbound syntax
* fix 15451

* add test cases

---------

Co-authored-by: Massimiliano Pippi <[email protected]>
fix `pip install` and `available_models` examples
…lama#15691)

* increase coverage for event.py

* ignore type checking imports for coverage

* properly shut down workflow in test

* cover workflow module and simplify validation
…#14652)

* local default model for embedding

* default model code for llm

* add default model code for reranker

* notebook rerun changes

* test cases update

* remove poetry lock files

* code cleanup

* update nvidia reranker notebook

* mock the integrate api calls

* mock integrate api calls in test case

* mock test cases in api_key

* add test cases for embeddings

* fix lint, test cases

* fix bug in rerank url parsing, test cases

* remove poetry lock files

* fix test cases, code

* fix lint issues

* less restrictive dep

* fix for the url warning and test cases

* add masked env to test case

* remove test case for now

---------

Co-authored-by: Logan <[email protected]>
…llama#15709)

* refact: merge Context and Session to simplify the workflows api

* add deprecation warnings

* remove session usage

* remove session usage from docs

* remove usage of self.data
* fix: stop streaming events when a step raises

* remove session
…utils/llama-index-utils-workflow (run-llama#15718)

chore(deps-dev): bump jupyterlab

Bumps [jupyterlab](https://github.com/jupyterlab/jupyterlab) from 4.2.4 to 4.2.5.
- [Release notes](https://github.com/jupyterlab/jupyterlab/releases)
- [Changelog](https://github.com/jupyterlab/jupyterlab/blob/@jupyterlab/[email protected]/CHANGELOG.md)
- [Commits](https://github.com/jupyterlab/jupyterlab/compare/@jupyterlab/[email protected]...@jupyterlab/[email protected])

---
updated-dependencies:
- dependency-name: jupyterlab
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…re (run-llama#15717)

Bumps [notebook](https://github.com/jupyter/notebook) from 7.2.1 to 7.2.2.
- [Release notes](https://github.com/jupyter/notebook/releases)
- [Changelog](https://github.com/jupyter/notebook/blob/@jupyter-notebook/[email protected]/CHANGELOG.md)
- [Commits](https://github.com/jupyter/notebook/compare/@jupyter-notebook/[email protected]...@jupyter-notebook/[email protected])

---
updated-dependencies:
- dependency-name: notebook
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
GeorgeDittmar and others added 30 commits September 9, 2024 17:21
* docs: update ai21 docs and readme with jamba 1.5 and tool calling

* docs: update ai21 docs and readme with jamba 1.5 and tool calling

* docs: update ai21 docs and readme with jamba 1.5 and tool calling
* feat(postgres): add support for engine parameters
- Introduced engine_params to support passing parameters to create_engine.
- Updated create_engine and create_async_engine calls to include engine_params.
- Initialized engine_params in the constructor.

* style(lint): reformat for readability

* refactor(postgres): rename engine_params to create_engine_kwargs

* refactor(postgres): rename engine_params to create_engine_kwargs

* chore: bump version to 0.2.3

* fix(postgres): rename engine_params to create_engine_kwargs
* update falkordb client

* bump version

* update version

* fix
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.