-
Notifications
You must be signed in to change notification settings - Fork 13
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
docs: deploy langfuse for development #347
Conversation
Note(s) for PR Auther:
Note(s) for PR Reviewer(s):
|
docs/langfuse.md
Outdated
|
||
### Set up Redis | ||
|
||
A local instance of LangFuse will bring it's own instance of `Redis` with some basic authentication setup. The password defaults to `myredissecret` (if you do not specify the env var `REDIS_AUTH`) and LangFuse will automatically use the database number `0` of that Redis instance (remember that it comes by default with 16 databases, 0 to 15). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A local instance of LangFuse will bring it's own instance of `Redis` with some basic authentication setup. The password defaults to `myredissecret` (if you do not specify the env var `REDIS_AUTH`) and LangFuse will automatically use the database number `0` of that Redis instance (remember that it comes by default with 16 databases, 0 to 15). | |
A local instance of LangFuse brings its own Redis instance with some basic authentication setup. The password defaults to `myredissecret` (if you do not specify the env var `REDIS_AUTH`) and LangFuse automatically uses the database number `0` of that Redis instance (remember that by default, it comes with 16 databases, 0 to 15). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could also link to LangFuse official docs in the first mention.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea, I will add this.
docs/langfuse.md
Outdated
|
||
A local instance of LangFuse will bring it's own instance of `Redis` with some basic authentication setup. The password defaults to `myredissecret` (if you do not specify the env var `REDIS_AUTH`) and LangFuse will automatically use the database number `0` of that Redis instance (remember that it comes by default with 16 databases, 0 to 15). | ||
If you also run a local instance of `Redis` for the `Companion`, you have two options. | ||
a) You let the `Companion` use the Redis instance that comes with Langfuse and addressing database `1` by adding these lines in your `config/config.json`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a) You let the `Companion` use the Redis instance that comes with Langfuse and addressing database `1` by adding these lines in your `config/config.json`: | |
1. You let the Companion use the Redis instance that comes with LangFuse and addressing database `1` by adding these lines in your `config/config.json`: |
To double-check if I get it right: the Redis instance comes with LangFuse and database 1
? What is it addressing? Did you mean "corresponding"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LangFuse comes with Redis. Redis has multiple Databases, 0 to 16. We want to set the Companion to use database 1
.
Maybe:
1. You let the Companion use the Redis instance that comes with LangFuse by addressing database `1` in `config/config.json`
Co-authored-by: Małgorzata Świeca <[email protected]>
Co-authored-by: Małgorzata Świeca <[email protected]>
Co-authored-by: Małgorzata Świeca <[email protected]>
Co-authored-by: Małgorzata Świeca <[email protected]>
Co-authored-by: Małgorzata Świeca <[email protected]>
Co-authored-by: Małgorzata Świeca <[email protected]>
Co-authored-by: Małgorzata Świeca <[email protected]>
Co-authored-by: Małgorzata Świeca <[email protected]>
67a52dd
to
93a521a
Compare
Description
Changes proposed in this pull request:
Related issue(s)
#49