-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
An expensive bug GPT4. no matter what I do #125
Comments
Thanks for reporting and sorry about the spend. It sounds like you did due diligence on debugging, so I’m not sure how it was charged to GPT-4. We’ve not seen it call the wrong models before, so I am very curious to try and replicate this. |
To help reproduce, would be great to see
|
Maybe here? Looks like the default model is set to GPT-4 Line 18 in 20d77a1
|
@LittleLittleCloud No I don't think that's it. That default is only accessed from the improve_code, improve_function, and implement calls -- and they aren't part of the main control path. I am eyeing this: autogen/autogen/oai/openai_utils.py Line 331 in 20d77a1
@twobob How are you loading the config list? Are you using .env? |
[ |
thanks for all the feedback. Yes there is zero reference to the word gpt-4 in my code now so |
me@mebuntu:~/repo/autogen$ echo $OPENAI_API_KEY my env is indeed setup as well as the config file |
hi , which code. its the default code gutted to not contain any references to gpt_4 pretty much? |
Could you show the code that contains |
I forgot to mention but I DID do this first on a windows box and wa using bash as my interpreter because "inline commands working for windows would be cool to do for everyone" I thought. That was when I noticed the bug. Thought it was /maaaaybe/ the funky bash hack that was causing the error Like this is the dream support box. literally nothing has happened on it other than this. since I have installed chatdev on the same box but meh. these tests were before that and nothing has changed in terms of the autogen anyway. So there. the full sordid details. Honestly not sure where to go from there It's not account since chatdev can use 3.5 just fine from the ubunutu install (why I installed chatdev after) |
hi. it is the defaults straight from the readme or from a notebook, does it write logs anywhere? Ive mounted onto the windows drive from ubuntu so I can access anything... |
i ran agentchat_groupchat_research.ipynb from the notebook folder in jupyter notebook in the beginning I was running it { and only the engineer was actually given gpt 4 access "gpt4_config = {\n",
|
Hmmm. you know I am certain I did NOT remove that one (I errantly excluded that from the replace)... |
So. i reran it just to make sure it wasnt anything really stupid. BEFORE I made the above change
so. I wasn't going mad it really was doing that. I made the above change. and only added 3.5 models in the list and ran it again/ Still no change. I attached my actual notebook. |
maybe I should remove the API key from env? Any other ideas? It is fair to say I am very sad this does not work hahahah |
attached the zip |
In the notebook I attached I had actually increased the engineer to 4 again. Hardly mattered I was already timing out on the gpt4 token/min before it popped |
The code looks good to me. Please make sure:
|
|
Ill do it after midnight. THEN I will for sure see it. it will be the only cost. lightbulb |
Ooooookay. It seems like maybe we have a suspect. config_list_from_json not actually populating ... hmm.. I DID NOT see that coming. good call. So... I guess time to make that env... Better find those docs I guess EDIT: https://microsoft.github.io/autogen/docs/FAQ/#set-your-api-endpoints I see |
I dont run this on colab sir. |
ok. then it should be located next to the .ipynb file. otherwise you need to specify |
Yup ^^^ This ^^^ I was NOT in the notebook. folder. It was one level below. Ill restest now this is all a bit more "like it should be" now when I print the config the keys are present. running test. There are 0 gpt3 cost for the day and I noted the gpt4 usage. Here's hoping! |
That sliver on blue IS A WELCOME SIGHT! Might I suggest a tweak to the notebooks? something that actually PRINTS or even CHECKS the content of that array and just refuses to continue otherwise with a warning? Thanks so much again Engineer (to chat_manager): As an AI language model, I don't have the capability to actively participate in the development process. However, I can provide guidance and suggestions based on the information provided. If you have any specific questions or need assistance with any particular step of the development process, please let me know and I'll be happy to help! How unfortunate... Ah well |
or try some non-coding tasks, or try including a code example in the initial message. |
Ok, here is my understanding of what's happening: If the OAI_CONFIG_LIST is not found, then the config that gets returned is an empty list, (see: autogen/autogen/oai/openai_utils.py Line 246 in 4522900
Then autogen/autogen/oai/completion.py Line 771 in 4522900
Finally OAI's openai library reads the missing key from environment variable. |
I set this up on a new machine today.
I realised that in 2 games I had suddenly spent 7 dollars instantly. Pretty steep considering it was supposed to be using 3.5-turbo-16k
Checked over all the setting. Then parsed the entire source code for ANY reference to default models. anything. Changed every single reference. No matter what I set it is using the GPT4 API endpoint.
I have tried; the usual inline declarations; using the built in preferred template; just jamming it in as the hardcoded value to the method that checks it. So.. unless anyone has any other ideas. I'm all out.
7 bucks is no big shakes but I run systems in parallel and no doubt others batch too. Wouldn't want anyone to get an AMAZON AWS level surprise and not have reported. so. Enjoy. Reported.
EDIT: It's a linux box but I did install it under windows on there as well and test it (that test is probably the top 50c of the green bar ) It was identical so "not OS". and "yes, I also tried that."
The text was updated successfully, but these errors were encountered: