search

langchain-ai / langsmith-docs

Documentation for langsmith
https://docs.smith.langchain.com/
MIT License
81 stars 37 forks source link

use langsmith env vars #373

Open baskaryan opened 3 months ago

baskaryan commented 3 months ago

Updates:

LANGCHAIN_TRACING_V2 -> LANGSMITH_TRACING
LANGCHAIN_PROJECT    -> LANGSMITH_PROJECT
LANGCHAIN_API_KEY    -> LANGSMITH_API_KEY
LANGCHAIN_ENDPOINT   -> LANGSMITH_ENDPOINT

not sure all of these actually work across py and js

vercel[bot] commented 3 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langsmith-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 29, 2024 10:56pm
baskaryan commented 3 months ago

pr doesn't touch:

LANGCHAIN_TEST_TRACKING
LANGCHAIN_TEST_CACHE
LANGCHAIN_HUB_API_KEY
LANGCHAIN_HIDE_INPUTS
LANGCHAIN_HIDE_OUTPUTS
LANGCHAIN_TRACING_SAMPLING_RATE
LANGCHAIN_HUB_API_URL

imo these should also be updated to use LANGSMITH prefix but seems like bigger change?

jacoblee93 commented 3 months ago

It looks like LANGSMITH_ENDPOINT, LANGSMITH_API_KEY, and LANGSMITH_PROJECT aren't in JS 😬

LANGSMITH_TRACING is at least.

agola11 commented 3 months ago

I would change in the SDK first, let it bake a while, then change docs later

agola11 commented 3 months ago

The changes in the SDK should bw-compat, the older env vars should still be respected