-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
SttTtsPlugin error when launching space #2206
Comments
This is crashing it:
Removing that log fixes it..im assuming something's off with the config? |
Hi @y4my4my4m I commented out the log at line 104 as suggested but still getting the same error. Any new findings? I am able to see the agent launched a space but keeps having connection issues. |
Did you correctly build after editing? |
Works after build, thanks. |
I'm having this error but the agent keep crashing after removing log and building Error: ICE connection failed [JanusClient] createRoom => {"janus":"success","session_id":439686258888454,"transaction":"7bdmxunc","sender":3795148812314350,"plugindata":{"plugin":"janus.plugin.videoroom","data":{"videoroom":"created","room":"1YpJklwYEPPxj","permanent":false}}} ◎ LOGS |
@CactusByte did you build? |
Hi @y4my4my4m, could you please provide steps to reproduce the issue? Could you check if the issue persists on the latest develop branch? I wasn’t able to reproduce it on my end. Thank you! |
@CactusByte I have been here before, it could be one of the things below.
suggest you add more error logs for startSpace function in spaces.ts |
one thing that is wrong with the spaces plugin is that it's reading the exported OPEN_API_KEY value from my zshrc instead of what's defined in the .env: // Retrieve keys
const openAiKey = process.env.OPENAI_API_KEY || "";
const elevenLabsKey = process.env.ELEVENLABS_XI_API_KEY || ""; should be like: runtime.getSetting("OPENAI_API_KEY") ?? instead. However that is not the issue as it works when i just comment out the config. I can't seem to run the latest dev branch, but on the latest release node v23.3.0 |
Got it thanks for clarifying 🫶 will check! |
someone reported this:
|
@tcm390 Im not getting any ICE issues. |
I'll try with the latest updates and let you know if the problem persists. But is there any specific requirement or tip i should consider to make it run correctly? What's your dev env? are you using the devcontainer? I am using a windows host and wsl. |
Still getting same error: Error: ICE connection failed Node.js v23.5.0 Also got this logs before the agent crashes: ◎ LOGS |
@worksgoodcompany |
related to this? #1965 |
Yeah i checked that one too, but still didn't solve the issue. i'm using ubuntu 22 on wsl, and tried with all the suggestions on that issue, but still having problems. my guess is that it's related to agent-twitter-client and @roamhq/wrtc. Any suggestion on how to make twitter spaces work without issues? |
Im having this same issue myself, i had it working about 1.5-2 weeks ago off the main branch, which is the strangest part. Perhaps a minor config difference that is hard to capture via a generic error? |
@Cooops my guess is that there's something that's trying to be printed in the "config" variable that doesn't exist. |
@y4my4my4m what the, yea just commented out that line and it worked -- back to talking with the space agent. so this log line causes issues when a certain config is missing, but we aren't sure which ones trigger it immediately based on the error alone? edit: ah, i think its the transcription provider (at least in my case...there's some interesting logic that happens re: checking there, as i do have one configured). if thats the culprit for most, which i think it might be, i can work on a smol fix tonight? |
I tried with the latest develop branch version and now spaces starts, i can join, but when i try talking i get this logs: [2025-01-22 23:04:36] ERROR: Max retry attempts reached: |
I was able to fix the previous ""invalid input syntax for type uuid:" error by making changes and improving handling of user ids and memory storage. Now i was able to make the spaces work, or "kinda" work, because is still not consistent. It seems the ICE connection error occurs when the agent creates a spaces for the first time and no one joins. If i join the spaces, this error doesn't happen and the agent doesn't crash. I'll keep on debuggin |
what did you change for improving handling of user ids and memory storage? Could you make a pr? |
Here is the PR with the changes i made #2686 |
Reviewed, thanks |
Describe the bug
To Reproduce
Current commit: d55c86c
Expected behavior
Launches the space without SttTtsPlugin crashing
The text was updated successfully, but these errors were encountered: