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

[Bug]: ollama output erroneously considers sample data #302

Open
gudlyf opened this issue Mar 6, 2024 · 2 comments
Open

[Bug]: ollama output erroneously considers sample data #302

gudlyf opened this issue Mar 6, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@gudlyf
Copy link

gudlyf commented Mar 6, 2024

Opencommit Version

3.0.11

Node Version

21.6.2

NPM Version

10.2.4

What OS are you seeing the problem on?

Mac

What happened?

When using ollama as the AI engine, it mistakenly considers the example git diff of server.ts in its output.

Example:

Changed port variable case and added support for configurable port in server.ts

Often times, it is the ONLY change it displays. It also ignores formatting, emoji, etc.

(In addition, the mistral LLM is hard-coded for some reason, when it should be possible to try others)

Expected Behavior

Output should look like it does with using OpenAI and not consider the sample input git diff.

Current Behavior

See "What happened."

Possible Solution

  • Possibly need to specify to ollama that the sample git diff is, indeed, a sample.
  • Ensure the correct git diff is considered, and in its entirety (it seems to skip many changes)
  • Consider a OC_OLLAMA_MODEL variable rather than hard-coded mistral LLM.

Steps to Reproduce

No response

Relevant log output

No response

@di-sukharev
Copy link
Owner

will be releasing next version this weekend, lets see if this is healed, thank you @digitalstudium <3

@di-sukharev
Copy link
Owner

done, 3.0.12 is out, please npm i -g opencommit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants