Message delete (when approaching tokens limit)

Ace-Radom commented 1 year ago

~我又来了~ 本次PR更改如下：

将 tokens_limit 改为 ChatGPT 类变量值更改现由 set_model 函数负责
在 ChatGPT 类内增加 delete_first_conversation 函数用于删除 messages 列表内的第一次对话【即 index=1,2 一问一答】
在使用的tokens接近限制时弹出提醒【默认是在剩余tokens在1~500时提醒后期可以改自定义】
~增加 /delfirst 命令用于删除目前的第一次对话【这个命名貌似不太好你要不看看改改或者干脆顺着这个写一个清空目前聊天记录的功能然后以和 /copy 命令一样的思路构建命令结构】~ 新commit已经完善为 /delete 命令
疑似bug修复：chat.py: 481, 482 中曾试图调用置于 ChatMode 类下的 timeout 变量但他是被定义在 ChatGPT 类下的我不知道为什么在老版本中进行了这样的修改但之前确实一旦调用 /timeout 命令就会直接报找不到对象退出因此将此处改回了 chatGPT.timeout 即：
```
if len(args) > 1:
new_timeout = args[1]
else:
new_timeout = prompt(
    "OpenAI API timeout: ", default=str(chatGPT.timeout), style=style)
#                                           ~~~~~~~
if new_timeout != str(chatGPT.timeout):
#                     ~~~~~~~
chatGPT.set_timeout(new_timeout)
else:
console.print("[dim]No change.")
```
但我不敢保证这样更改是正确的你能不能再看一下

另外报告一个问题：

/usage 命令目前疑似失效了 https://api.openai.com/dashboard/billing/credit_grants 这个url貌似现在阻止了非浏览器的一切访问之前问ChatGPT他给出了一样的答复不知道这样的话usage统计功能是否要重写

Ace-Radom commented 1 year ago

顺便 delete_first_conversation 函数的代码可能需要优化一下但我这个写C的实在是不知道有没有更优雅的写法了~毕竟没系统学过python~

Ace-Radom commented 1 year ago

又改了点东西

delete_first_conversation 函数优化过了从两连pop改为了用del删除
将 /delfirst 命令完善为 /delete 命令可选二级命令为 first 和 all 用于删除第一次对话或是所有对话在没有二级命令时默认删除第一次对话
在help page里加入了对 /delete 命令的描述

xiaoxx970 commented 1 year ago

可以的很好，timeout确实忘了，当时改是因为我发现timeout这个变量只在CHATGPT类里用到，所以就移过来了，ChatMode就专门放各种模式。然后我还把其他地方的chatGPT改成chat_gpt了匹配python的命名方法，这个handle_command里的名字忘了改，你有空也可以改改，把chatGPT都改成chat_gpt 还有就是yes/no确认这个可以直接用prompt_toolkit.shortcuts的confirm，直接放到下面的提示确认prompt那里就行

xiaoxx970 commented 1 year ago

/usage命令确实失效了，那个api没在文档里，所以不让用了也能理解，在文档里的只有/v1/usage接口，但这个是查api的使用记录，没有费用信息。所以现在这样显示none也行，或者暂时不显示这个命令，或者在用的时候给个usage页面的链接

Ace-Radom commented 1 year ago

那我个人觉得就隐藏掉 /usage 命令吧毕竟目前估计是没有方法能在官方外查到余额了但不完全删除因为后期可以再摸索一下有没有别的方法

Ace-Radom commented 1 year ago

我删掉了 /usage 命令的自动补全和帮助页的描述但保留了原来的功能实现代码并且在那里加了一行提示 (chat.py: 425)

console.print("[red]`[bright_magenta]/usage[/]` command is currently unavailable, it's not sure if this command will be available again or not.")

不知道日后能不能找到别的绕过浏览器识别的方法现在那个api不让用就是因为他强制必须是浏览器访问才可以貌似但这方面我不是非常清楚只是他报错这么说你看要么按照这样改一下README

xiaoxx970 commented 1 year ago

OK完事了

xiaoxx970 / chatgpt-in-terminal

Message delete (when approaching tokens limit) #14