-
Notifications
You must be signed in to change notification settings - Fork 858
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
add SKIP_TRIGGER_KEY_WORD, SKIP_SEND_TO_OpenAI_KEYWORDS #36
Conversation
可以,我晚上看下,下午要干活了 |
我还有一个想法,是添加场景定制的,之前我自己定制的就是比如给一个5岁的小孩用, 这样会更有上下文一些。 |
6ae1508
to
625c98e
Compare
@pjq 定制场景的我们得设计下。你先开个 issue? |
嗯 |
38818a1
to
b963845
Compare
Test done. |
今天先睡了,明天看哈。邀请你作为合作者了。 |
顺序是先合 config -> 然后我测一下这个 PR -> 最后我们一起看看那个提示词 和 #42 |
958369f
to
d769cb4
Compare
Test the following two cases, works well.
|
e6d6c82
to
ac3ebde
Compare
@yihong0618 目前新加的black . --check -v 格式化之后还是通过不了 |
升级你本地的 black pip install -U black |
b7fad1d
to
5adf2f8
Compare
Fix the code conflicts
|
will take a look later |
will merge and seems we need to refactor the code in the future. |
ENABLE_IMMERSIVE_TALKING_MODE need to set false when we use arg I wonder |
Now it's kinds of mess up for the arguments/config/environment/constants. |
yep... |
add more config field to the config file rename the fields name fix PROMPT config add CLI_INTERACTIVE_MODE params
等我研究下把这个合了,整理代码。。。 |
@pjq closing this since we have supported key words now |
添加两个参数
@yihong0618 你看看,需不需要