amica icon indicating copy to clipboard operation
amica copied to clipboard

Ollama Patch for Fixing Potential Edge Case Issues (Issue Renamed)

Open orsonteodoro opened this issue 1 year ago • 4 comments

Commit da5a390 breaks Ollama support.

Tested with Ollama 0.3.14 and 0.4.2.

Reverted commit makes it work.

orsonteodoro avatar Nov 18 '24 15:11 orsonteodoro

For a possible fix for support for Ollama api/chat see https://github.com/orsonteodoro/oiledmachine-overlay/blob/master/app-misc/amica/files/amica-0.2.1_p20241022-ollama.patch

The patch may need to be fixed a little.

orsonteodoro avatar Nov 29 '24 05:11 orsonteodoro

Thanks for the link. We'll look into it

slowsynapse avatar Nov 29 '24 07:11 slowsynapse

Hello @orsonteodoro we tried to recreate the problem, but it works fine on: 0.51, 0.3.14 and 0.4.2

Thanks for the patch write. Still looking into it. : )

slowsynapse avatar Dec 09 '24 08:12 slowsynapse

I have renamed the issue, since it's now about whether we patch it or not to streamline the chunking.

slowsynapse avatar Dec 18 '24 14:12 slowsynapse