deps-managment-and-dotenv #1

Open
EndMove wants to merge 3 commits from EndMove/darknet_diaries_llm:deps-managment-and-dotenv into master
First-time contributor

Add deps managment and dotenv to load env vars

Add deps managment and dotenv to load env vars
EndMove added 2 commits 2023-10-07 16:57:12 +02:00
phito requested changes 2023-10-09 09:52:13 +02:00
@ -7,3 +7,2 @@
if not os.path.exists(folder_path):
os.makedirs(folder_path)
if __name__ == '__main__':
Owner

c'est vraiment nécessaire d'avoir ça dans un fichier qui n'est pas importé par d'autres fichiers? C'est juste un script pas un module

c'est vraiment nécessaire d'avoir ça dans un fichier qui n'est pas importé par d'autres fichiers? C'est juste un script pas un module
Author
First-time contributor

yup pour spécifier que c'est un script "executable"

yup pour spécifier que c'est un script "executable"
@ -14,1 +15,3 @@
llm = OpenAI(model="gpt-4", temperature=0, max_tokens=256)
# load .env
load_dotenv()
OPEN_API_KEY = os.getenv('OPEN_API_KEY')
Owner

pas nécessaire, llama-index prend déjà la clé API depuis les envvar

pas nécessaire, llama-index prend déjà la clé API depuis les envvar
Author
First-time contributor

pour ceux qui ne passent pas par la :p genre moi ^^

pour ceux qui ne passent pas par la :p genre moi ^^
main.py Outdated
@ -15,0 +17,4 @@
OPEN_API_KEY = os.getenv('OPEN_API_KEY')
# config llm context
llm = OpenAI(model="gpt-4", temperature=0, max_tokens=256, api_key="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")
Owner

merci pour la clé gratuite xD

merci pour la clé gratuite xD
Author
First-time contributor

oupss ;')

oupss ;')
@ -34,3 +24,1 @@
}
)
documents.append(document)
# TODO split in small functions
Owner

no

no
EndMove added 1 commit 2023-10-09 10:07:27 +02:00
Author
First-time contributor

@phito j'ai up :D

@phito j'ai up :D
This pull request can be merged automatically.
You are not authorized to merge this pull request.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u deps-managment-and-dotenv:EndMove-deps-managment-and-dotenv
git checkout EndMove-deps-managment-and-dotenv

Merge

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff EndMove-deps-managment-and-dotenv
git checkout master
git merge --ff-only EndMove-deps-managment-and-dotenv
git checkout EndMove-deps-managment-and-dotenv
git rebase master
git checkout master
git merge --no-ff EndMove-deps-managment-and-dotenv
git checkout master
git merge --squash EndMove-deps-managment-and-dotenv
git checkout master
git merge --ff-only EndMove-deps-managment-and-dotenv
git checkout master
git merge EndMove-deps-managment-and-dotenv
git push origin master
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: phito/darknet_diaries_llm#1
No description provided.