Skip to content
This repository was archived by the owner on Oct 22, 2023. It is now read-only.

Supports gpt3.5 turbo chat instead of the davinci completion api. Added a simple client that can be used locally.#12

Merged
lancecarlson merged 3 commits intodaveshap:mainfrom
lancecarlson:main
Apr 26, 2023
Merged

Supports gpt3.5 turbo chat instead of the davinci completion api. Added a simple client that can be used locally.#12
lancecarlson merged 3 commits intodaveshap:mainfrom
lancecarlson:main

Conversation

@lancecarlson
Copy link
Collaborator

No description provided.

@alfredo-f
Copy link
Contributor

Hi @lancecarlson,

I opened #13, which includes a more comprehensive refactoring using Black.

How does it look to you?

@alfredo-f
Copy link
Contributor

@lancecarlson

I think this pull request is bringing together very different modifications in a single shot.

I think we should be aiming first at

  • cleaning up the repository structure
  • add basic tests

I'm working on both of them right now, and I'll be opening pull requests soon.

What do you think?

@lancecarlson
Copy link
Collaborator Author

Hi @alfredo-f Yes. I had intended to just get this PR quickly pulled before making more changes. @daveshap Do you think we could pull this PR in first and then work on the other suggested changes that @alfredo-f was considering?

@daveshap
Copy link
Owner

I don't have the mental bandwidth to manage an OSS project, so I need y'all to discuss it amongst yourselves. You are both doing good work so I trust you to work it out and align on a correct order of operations. If you could collaborate on Discord on the Discussion tab, that would help a tremendous amount. Let me know if you need help coordinating.

@alfredo-f
Copy link
Contributor

I don't have the mental bandwidth to manage an OSS project, so I need y'all to discuss it amongst yourselves. You are both doing good work so I trust you to work it out and align on a correct order of operations.

@daveshap we need write access to merge without bothering you every time

If you could collaborate on Discord on the Discussion tab, that would help a tremendous amount.

@daveshap we still need this platform not to decouple code from discussion

I had intended to just get this PR quickly pulled before making more changes.

@lancecarlson sounds good to me, we can merge and take it from there

@lancecarlson lancecarlson merged commit 044d913 into daveshap:main Apr 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants