opencommit icon indicating copy to clipboard operation
opencommit copied to clipboard

[Bug]: ollama output erroneously considers sample data

Open gudlyf opened this issue 1 year ago • 2 comments
trafficstars

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

gudlyf avatar Mar 06 '24 21:03 gudlyf

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

di-sukharev avatar Mar 15 '24 07:03 di-sukharev

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

di-sukharev avatar Mar 18 '24 07:03 di-sukharev