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

Add example of .env file #59

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

pacexy
Copy link

@pacexy pacexy commented Jul 25, 2023

This will make it easier for users to set up environment variables.

@lingchuL
Copy link

lingchuL commented Jul 25, 2023

目前代码里是按是否有相应配置来决定要使用OpenAI还是Azure的,并且在检测到有OpenAI的配置会直接使用OpenAI的API尝试,所以如果只是添加用例可能把几行都默认为注释状态比较好~
In the createLanguageModel process, which endpoint to use is determined by whether there is a corresponding configuration to them. It will try to get OpenAI env variable first, once it get the OPENAI_API_KEY, it will use OpenAI as endpoint. Therefore, it may be better to comment all those lines as the default state~

@pacexy
Copy link
Author

pacexy commented Jul 26, 2023

目前代码里是按是否有相应配置来决定要使用OpenAI还是Azure的,并且在检测到有OpenAI的配置会直接使用OpenAI的API尝试,所以如果只是添加用例可能把几行都默认为注释状态比较好~ In the createLanguageModel process, which endpoint to use is determined by whether there is a corresponding configuration to them. It will try to get OpenAI env variable first, once it get the OPENAI_API_KEY, it will use OpenAI as endpoint. Therefore, it may be better to comment all those lines as the default state~

Maybe we can just keep it empty? (e.g. OPENAI_API_KEY=), and then it will not affect the createLanguageModel.

@lingchuL
Copy link

Yeah, it works too.

@Stevenic
Copy link

Stevenic commented Aug 9, 2023

You don't normally include the "" quotes in .env.example files. empty is better

@pacexy
Copy link
Author

pacexy commented Oct 18, 2023

You don't normally include the "" quotes in .env.example files. empty is better

Updated.

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.

3 participants