Skip to content
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

Refactor PI function into shared library #317

Closed
wants to merge 24 commits into from
Closed

Conversation

neko-para
Copy link
Contributor

No description provided.

@neko-para neko-para mentioned this pull request Aug 25, 2024
25 tasks
@neko-para
Copy link
Contributor Author

@MistEO 有个设计上的问题感觉可以讨论一下

目前, 我整体接口的设计逻辑依然是把PI处理部分作为一个黑盒来提供; 对于使用PI库的人来说, 实际就是给PI提供一个界面前端(用来实现功能选择, 一个回调之类的)

但是这样的话, 这个库本身的价值就很有限, 并不太能用来给集成模式使用; 是不是可以看看扩展一下PI的功能, 使得集成模式依然有使用的价值? 目前我想到的是可以把创建实例部分和执行task拆开, 自己集成的情况下依然可以用PI来快速创建实例, 以及其所涉及到的用户选择部分的逻辑(而不需要集成方自己实现); 但是后续的task执行和option部分好像还是不太好复用

@neko-para
Copy link
Contributor Author

或者就是单纯让使用PI库的一方自己去编辑config, 但感觉就比较奇怪了(或者说, 要着库有啥用啊那

@MistEO
Copy link
Member

MistEO commented Aug 27, 2024

@MistEO 有个设计上的问题感觉可以讨论一下

目前, 我整体接口的设计逻辑依然是把PI处理部分作为一个黑盒来提供; 对于使用PI库的人来说, 实际就是给PI提供一个界面前端(用来实现功能选择, 一个回调之类的)

但是这样的话, 这个库本身的价值就很有限, 并不太能用来给集成模式使用; 是不是可以看看扩展一下PI的功能, 使得集成模式依然有使用的价值? 目前我想到的是可以把创建实例部分和执行task拆开, 自己集成的情况下依然可以用PI来快速创建实例, 以及其所涉及到的用户选择部分的逻辑(而不需要集成方自己实现); 但是后续的task执行和option部分好像还是不太好复用

主要是我们自己复用吧,比如 vscode 插件可以用、maa debugger 可以用,免得每边造一个轮子

能自己写 UI 的人我感觉也不怎么需要这么一个 config 库(

@neko-para
Copy link
Contributor Author

那接口还是就按照 黑盒 + 输入回调 的模型来做了?目前在糊interface的解析逻辑的时候顺便把国际化做了(#244),但是这个和输入回调的相性不是很好来着(
以及,如果输入回调只是简单的提供prompt和choice,实际操作起来其实相当于限制死了ui是以若干个选择来进行的,不一定是个好主意(比如debugger里面如果想做成下拉框就很麻烦

@neko-para
Copy link
Contributor Author

生成默认配置

QQ_1724847361181

@neko-para neko-para requested a review from MistEO August 28, 2024 12:22
Base automatically changed from refactor/exec_agent to dev September 2, 2024 05:42
Base automatically changed from dev to main September 13, 2024 10:20
@neko-para neko-para closed this Sep 17, 2024
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.

4 participants