Skip to content

Studocs#15

Closed
thestumonkey wants to merge 2 commits intoSimpleOpenSoftware:mainfrom
thestumonkey:studocs
Closed

Studocs#15
thestumonkey wants to merge 2 commits intoSimpleOpenSoftware:mainfrom
thestumonkey:studocs

Conversation

@thestumonkey
Copy link
Contributor

I wanted to add further config flexibility, so added support to use other LLMs such as openAI.
I changed some env vars to suit this and added a seperate mem0 client, but looks like there is a new service.
I added support to use mem4j using graph memorty, but that's not quite working.

documented all the env vars.

Added docs, amended .env.template, moved the mem0 client to a seperate file and enabled graph memory. Renamed friend-lite to omi-lite for containers.
# Conflicts:
#	backends/advanced-backend/.dockerignore
#	backends/advanced-backend/Dockerfile
#	backends/advanced-backend/docker-compose.yml
#	backends/advanced-backend/src/main.py
@skywinder
Copy link
Contributor

loveley! great updates!

### Tunelling
We need to route an external domain to our backend. You can either use NGROK or Cloudflared to do this.
By default, there's an NGROK docker image, so if you use this you need
`NGROK_URL=yourngrokurl.app`
Copy link
Contributor

Choose a reason for hiding this comment

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

where is this NGROK_URL using in the code? cand find any references

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I thought this was needed to connect to the backend from the app, but I think this is done via expo, so I guess not needed

"uvicorn>=0.34.2",
"wyoming>=1.6.1",
"aiohttp>=3.8.0",
"deepgram-sdk>=4.0.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This shouldn't be required because its already in its own group

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I couldn't get this to work with the groups, but likely my issue

Copy link
Collaborator

@AnkushMalaker AnkushMalaker left a comment

Choose a reason for hiding this comment

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

leaving some WIP comments as I keep reviewing your WIP work.
Thanks!

Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't thin this file should exist, because this isn't a package?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added this when I was importing a mem0 module, but the memory service been refacvtored since then so can be removed

@@ -1,18 +1,14 @@
services:
friend-backend:
omi-lite:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this required? Doesn't quite matter to me but do think it makes sense repo is called friend-lite and this said friend-backend?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I renamed it to omi-lite as friend is no longer, but up to you.

@thestumonkey
Copy link
Contributor Author

made a new PR #22

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.

3 participants