phito/darknet_diaries_llm
2
0
Fork 1
Code Issues Pull Requests 1 Packages Projects Releases Wiki Activity

deps-managment-and-dotenv #1

Open
EndMove wants to merge 3 commits from EndMove/darknet_diaries_llm:deps-managment-and-dotenv into master
pull from: EndMove/darknet_diaries_llm:deps-managment-and-dotenv
merge into: phito:master
Conversation 2 Commits 3 Files Changed 6 +157 -108
EndMove commented 2023-10-07 16:57:11 +02:00
First-time contributor
Copy Link

Add deps managment and dotenv to load env vars

Add deps managment and dotenv to load env vars
🚀 1
EndMove added 2 commits 2023-10-07 16:57:12 +02:00
feat: package manager 0730db5727
feat: ups deps, add dotenv 6d7600e47e
phito requested changes 2023-10-09 09:52:13 +02:00
download_transcripts.py
@@ -7,3 +7,2 @@
if not os.path.exists(folder_path):
os.makedirs(folder_path)
if __name__ == '__main__':
phito commented 2023-10-07 18:13:10 +02:00
Owner
Copy Link

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
EndMove commented 2023-10-09 09:58:06 +02:00
Author
First-time contributor
Copy Link

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

yup pour spécifier que c'est un script "executable"
👍 1
main.py
@@ -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')
phito commented 2023-10-07 18:05:49 +02:00
Owner
Copy Link

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
EndMove commented 2023-10-09 10:02:45 +02:00
Author
First-time contributor
Copy Link

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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")
phito commented 2023-10-07 18:06:22 +02:00
Owner
Copy Link

merci pour la clé gratuite xD

merci pour la clé gratuite xD
EndMove commented 2023-10-09 09:58:14 +02:00
Author
First-time contributor
Copy Link

oupss ;')

oupss ;')
❤️ 1
main.py
@@ -34,3 +24,1 @@
}
)
documents.append(document)
# TODO split in small functions
phito commented 2023-10-07 18:06:51 +02:00
Owner
Copy Link

no

no
😆 1
EndMove added 1 commit 2023-10-09 10:07:27 +02:00
fix: pr comments 7774fc2698
EndMove commented 2023-10-09 10:07:51 +02:00
Author
First-time contributor
Copy Link

@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.
View command line instructions

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
Sign in to join this conversation.
Reviewers
No Reviewers
phito
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
phito
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: phito/darknet_diaries_llm#1
Delete Branch "EndMove/darknet_diaries_llm:deps-managment-and-dotenv"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?