xiaoxx970 / chatgpt-in-terminal

Use ChatGPT in terminal
MIT License
201 stars 27 forks source link

feat: 用于列出目前各个设置的值的命令 #39

Open Ace-Radom opened 1 year ago

Ace-Radom commented 1 year ago

隔壁重构计划基本功能achieve啦 虽然没能用纯C完成 但也没有像刚开始想的那样自己造一堆轮子 只自己写了个富文本库和markdown解析器 SSE事件用libcurl也直接解决了 总之还是挺顺利的 你有兴趣可以去看下ww


这也是一个我加在重构计划里的新功能

image

总之就是用一个命令直接打印出目前所有设置项的值 我用的是 /list 感觉在有人忘了目前比如说timeout设的多少或者temperature设的多少直接用这个命令就能看 你可以考虑一下【考虑到i18n也在写所以就没直接开pr 这个实现起来也应该很简单】

xiaoxx970 commented 1 year ago

隔壁重构计划基本功能achieve啦 虽然没能用纯C完成 但也没有像刚开始想的那样自己造一堆轮子 只自己写了个富文本库和markdown解析器 SSE事件用libcurl也直接解决了 总之还是挺顺利的 你有兴趣可以去看下ww

恭喜恭喜~等有空我下载下来体验看看

打印setting我也是觉得有必要的,我也会忘了现在设置是啥,然后我会直接运行比如/timeout然后看到数值后Crtl-C。 可以直接叫/settings,不然list感觉会有歧义,或者/list_settings?这样又太长了

Ace-Radom commented 1 year ago

恭喜恭喜~等有空我下载下来体验看看

谢谢ww 虽然这个东西现在还只支持在linux下运行但我准备在一些新东西完善好之后移植到其余两个平台上【我毕竟也是主用win的人为了跑这个程序开个wsl多少有点hh】

打印setting我也是觉得有必要的,我也会忘了现在设置是啥,然后我会直接运行比如/timeout然后看到数值后Crtl-C。 可以直接叫/settings,不然list感觉会有歧义,或者/list_settings?这样又太长了

我觉得 /settings 也不错 或者干脆 /ls 虽然和系统命令重叠了就是


绷不住了隔壁这v1.0.0发布了就发现恶性bug给我弄的一头雾水 貌似是优化的锅 勉强绕过去了之后又发现ubuntu里的行为和debian里不一致 真的醉了 反正任重道远了 还有一堆的bug等着去修

xiaoxx970 commented 1 year ago

绷不住了隔壁这v1.0.0发布了就发现恶性bug给我弄的一头雾水 貌似是优化的锅 勉强绕过去了之后又发现ubuntu里的行为和debian里不一致 真的醉了 反正任重道远了 还有一堆的bug等着去修

加油小伙子😁

Lemon-2333 commented 1 year ago

需要我来加这个命令吗,顺便也提取文本了。 为什么命令不叫/config或者/info

xiaoxx970 commented 1 year ago

需要我来加这个命令吗,顺便也提取文本了。 为什么命令不叫/config或者/info

也可以,直观然后没有歧义就行。config或者settings更直观一点我觉得。

可以先把那个i18n的分支合并后再写这个,或者创建一个单独的pr,避免混乱

Lemon-2333 commented 1 year ago

需要我来加这个命令吗,顺便也提取文本了。 为什么命令不叫/config或者/info

也可以,直观然后没有歧义就行。config或者settings更直观一点我觉得。

可以先把那个i18n的分支合并后再写这个,或者创建一个单独的pr,避免混乱

ok,那么具体要展示哪些东西,以及配色怎么办(美术色盲 还有一样东西是用:Temperature还是用Randomness。还是Randomness/Temperature

Lemon-2333 commented 1 year ago

需要我来加这个命令吗,顺便也提取文本了。为什么命令不叫/config或者/info呢>> 也可以,直观然后没有歧义就行。config或者settings更直观一点我觉得。>>可以先把那个i18n的分支合并后再写这个,或者创建一个单独的pr,避免混乱我

i18n分支的Pr我提交了

xiaoxx970 commented 1 year ago

需要我来加这个命令吗,顺便也提取文本了。 为什么命令不叫/config或者/info

也可以,直观然后没有歧义就行。config或者settings更直观一点我觉得。 可以先把那个i18n的分支合并后再写这个,或者创建一个单独的pr,避免混乱

ok,那么具体要展示哪些东西,以及配色怎么办(美术色盲 还有一样东西是用:Temperature还是用Randomness。还是Randomness/Temperature

用Temperature,显示就无所谓单词长短了尽量和api一致。展示的话能用斜杠命令修改的配置都展示吧。先不用展示配置文件里的设置,等 #36 完成后再考虑。

Ace-Radom commented 1 year ago

最近在忙活自己的shell-like应用框架ww 内核搭完了最近 这几天我把这个功能写一下放松放松吧 命令我就用 /config 了 先实现一下基本的list功能 到时候PR开出来再一起想想要实现哪些别的要不 你看看怎么样

xiaoxx970 commented 1 year ago

最近在忙活自己的shell-like应用框架ww 内核搭完了最近 这几天我把这个功能写一下放松放松吧 命令我就用 /config 了 先实现一下基本的list功能 到时候PR开出来再一起想想要实现哪些别的要不 你看看怎么样

可以可以,你尽管弄吧