ecoute icon indicating copy to clipboard operation
ecoute copied to clipboard

Added reverse engineered ChatGPT API

Open lunjielee opened this issue 1 year ago • 4 comments

Hi, @SevaSk,

I hope you're doing well. I wanted to let you know about a PR that I've submitted. It introduces a new feature to ecoute, allowing users to utilize it without the need for a paid OpenAI account. Could you please take a look and provide your feedback?

Thanks, Lunjie

lunjielee avatar Jun 10 '23 04:06 lunjielee

Hmm not sure its OK to merge since this is probably not legal. Also for design I prefer to follow the tell, don't ask principle. Here why are we asking if the revChatGPT flag is set every time we want generate a response? Wouldn't it be better to tell the class to override the different response generation functionally once at the beginning of the program instead? No need to complicate class with more flags.

SevaSk avatar Jun 10 '23 17:06 SevaSk

Hmm not sure its OK to merge since this is probably not legal. Also for design I prefer to follow the tell, don't ask principle. Here why are we asking if the revChatGPT flag is set every time we want generate a response? Wouldn't it be better to tell the class to override the different response generation functionally once at the beginning of the program instead? No need to complicate class with more flags.

With regards to the first question, are you stating that merging the code is prohibited due to license issues or the utilization of a reverse-engineered API?

Regarding the second question, you are absolutely right. It is preferable to adhere to the "tell, don't ask" principle. I will ensure to modify that section of the code accordingly.

Lunjie

lunjielee avatar Jun 10 '23 21:06 lunjielee

Hmm not sure its OK to merge since this is probably not legal. Also for design I prefer to follow the tell, don't ask principle. Here why are we asking if the revChatGPT flag is set every time we want generate a response? Wouldn't it be better to tell the class to override the different response generation functionally once at the beginning of the program instead? No need to complicate class with more flags.

Lunjie Hi Lunjie I downloaded your files and replaced them with the original files however I am getting an error "ImportError: cannot import name 'ACCESS_TOKEN' from 'keys' (C:\Users\yahsk\Desktop\genesis\ecoute\ecoute\keys.py)" With the original files it was working properly and was facing no issues. I have the keys.py file in the folder with my openAI access key and am calling "python main.py --revChatGPT" Capture

Rakshay-pawar avatar Dec 26 '23 22:12 Rakshay-pawar

Hi, @SevaSk,

I hope you're doing well. I wanted to let you know about a PR that I've submitted. It introduces a new feature to ecoute, allowing users to utilize it without the need for a paid OpenAI account. Could you please take a look and provide your feedback?

Thanks, Lunjie

Lunjie Hi Lunjie I downloaded your files and replaced them with the original files however I am getting an error "ImportError: cannot import name 'ACCESS_TOKEN' from 'keys' (C:\Users\yahsk\Desktop\genesis\ecoute\ecoute\keys.py)" With the original files it was working properly and was facing no issues. I have the keys.py file in the folder with my openAI access key and am calling "python main.py --revChatGPT"

Rakshay-pawar avatar Dec 26 '23 22:12 Rakshay-pawar