-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
added support for automatic_label_demo.py without chatgpt #142
Conversation
@lsch0lz Thanks for your commit! I also try to use wget https://raw.githubusercontent.com/nltk/nltk_data/gh-pages/packages/tokenizers/punkt.zip
# unzip in '~/nltk_data/tokenizers'
wget https://raw.githubusercontent.com/nltk/nltk_data/gh-pages/packages/taggers/averaged_perceptron_tagger.zip
# unzip in '~/nltk_data/taggers'
wget https://raw.githubusercontent.com/nltk/nltk_data/gh-pages/packages/corpora/wordnet.zip
# unzip in 'nltk_data/corpora' |
@Andy1621 Thanks for the Feedback! |
Also what do you mean by:
If I understand the intention correct |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Good To Test 🛠
@lsch0lz Thanks! For However, using |
Thanks for this PR, I'm going to merge it! |
Hi! Since it's only possible to use
automatic_label_demo.py
with a paid OpenAI Account I implemented a method to use the functionality without ChatGPT.To achieve this I used NLTK to extract nouns from the caption. I also used a Lemmatizer to improve the produced tags.
Your implementation with ChatGPT:
My implementation without ChatGPT:
I implemented the feature in way to let the user choose if they want to use ChatGPT or not. If the script get startet without the
openai_key
by default NLTK will be used. If the user provides a valid key ChatGPT will be used.Let me know what you think about this feature.