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.

Sign up for GitHub

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

feat(llm-observability): add langchain integration #159

Merged
merged 11 commits into from
Jan 13, 2025
Merged

feat(llm-observability): add langchain integration #159

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
945d67d
feat(ai): LangChain integration v0.1
skoob13 Jan 10, 2025
2d6513e
test: langchain integration tests
skoob13 Jan 11, 2025
f629591
test: langchain-openai for v2 and v3
skoob13 Jan 13, 2025
d1fa606
chore: reorganize imports
skoob13 Jan 13, 2025
d8e2a50
fix: ci
skoob13 Jan 13, 2025
6111eff
fix: set python on ci to 3.9
skoob13 Jan 13, 2025
abc08a7
fix: upgrade ci for python 3.9
skoob13 Jan 13, 2025
3f0f352
fix: fallback for distinct_id
skoob13 Jan 13, 2025
9f7e094
fix: personless events for omitted distinct_ids
skoob13 Jan 13, 2025
739c88a
fix: review comments
skoob13 Jan 13, 2025
fff0a3f
feat: base url retrieval
skoob13 Jan 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ jobs:
- name: Check formatting with black
run: |
black --check .

- name: Lint with flake8
run: |
flake8 posthog --ignore E501
Expand All @@ -51,10 +51,10 @@ jobs:
with:
fetch-depth: 1

- name: Set up Python 3.8
Copy link
Member

Choose a reason for hiding this comment

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

Why the Python version change? Just wanna be sure we aren't silently dropping older Python versions. Though 3.8 should be fine to drop as it's EoL – we should just be explicit about that event if it happens. (To be honest also surprised we don't use a matrix of Python versions for this Actions job)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

langchain-community requires Python >=3.9, so the CI was failing on 3.8. It is listed under test requirements, so no package will be installed for people not using the AI integration.

- name: Set up Python 3.9
uses: actions/setup-python@v2
with:
python-version: 3.8
python-version: 3.9

- name: Install requirements.txt dependencies with pip
run: |
Expand Down
3 changes: 2 additions & 1 deletion .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,5 @@ posthog-analytics
.idea
.python-version
.coverage
pyrightconfig.json
pyrightconfig.json
.env
2 changes: 1 addition & 1 deletion llm_observability_examples.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
import uuid

import posthog
from posthog.ai import AsyncOpenAI, OpenAI
from posthog.ai.openai import AsyncOpenAI, OpenAI

# Example credentials - replace these with your own or use environment variables
posthog.project_api_key = os.getenv("POSTHOG_PROJECT_API_KEY", "your-project-api-key")
Expand Down
4 changes: 0 additions & 4 deletions posthog/ai/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +0,0 @@
from .providers.openai.openai import OpenAI
from .providers.openai.openai_async import AsyncOpenAI

__all__ = ["OpenAI", "AsyncOpenAI"]
3 changes: 3 additions & 0 deletions posthog/ai/langchain/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
from .callbacks import PosthogCallbackHandler

__all__ = ["PosthogCallbackHandler"]
Loading
Loading