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

Update packages and fix streaming from gpt4 model #100

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sparrovv
Copy link

@sparrovv sparrovv commented Feb 8, 2024

Main reason:

  • streaming GPT-4 started failing b/c sse.js was outdated and had bugs that caused unpredictable failures.

Main reason:

- streaming GPT-4 started failing b/c sse.js was outdated and had bugs
  that caused unpredictable failures.
@sparrovv
Copy link
Author

@bramses could you look into it?

@nicu-chiciuc
Copy link

@sparrovv Thanks for taking the time to fix this. I've been using this extension for multiple months now, and since it broke, I wasn't sure what to switch to, and was too lazy to actually come in and fix the code.

It appears that the project doesn't get too much attention for some time now, but it would be great if it did since it creates an amazing workflow.

If this PR doesn't get merged, and you decide to publish your fork, please let me know, I would be ready to contribute a little bit if I know somebody else is also interested in this.

@sparrovv
Copy link
Author

hey @nicu-chiciuc , thanks. I like this plugin a lot, so that's why I took the effort.

I'm not planning to publish my fork atm, as I wouldn't be able to maintain it properly. Hopefully someone with more time will get this moving.

@complyue
Copy link

ping @bramses ?

Love this plugin, maybe migrate it into some org, then trust some contributors so they can make fixes & publish new versions?

@Aurelian-Shuttleworth
Copy link

ping @bramses ?

Love this plugin, maybe migrate it into some org, then trust some contributors so they can make fixes & publish new versions?

Agreed, this would be the way! I find this tool far too useful to leave possibly unmaintained.

@bramses
Copy link
Owner

bramses commented Apr 30, 2024

Hey all! Apologies for the radio silence on this, I have been under the false impression to myself that I would get back to ChatGPT MD at some point, but I am now sure that won't be the case due to a variety of factors I'd be happy to discuss elsewhere, but I digress! Much more importantly, I sincerely apologize as it has left all of you in the lurch, and that's not cool of me!

Love this plugin, maybe migrate it into some org, then trust some contributors so they can make fixes & publish new versions?

This is probably the best solution, is there any org or maintainer who would be willing? Would be happy to discuss rollover plans.

Speak soon!


P.S. As to the issue at hand, it is likely due to a change in the SSE spec from OpenAI's models, perhaps? Not 100% sure.

@collinmesser
Copy link

collinmesser commented Jun 8, 2024

@bramses I may be willing to help maintain here too. Especially if there are others willing to help out, maybe @nicu-chiciuc ? I'm new to obsidian plugins but it'd be fun to work on occasionally, and I've done a bit of open source maintaining in the past.

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.

None yet

6 participants