Ollama Integration

[AlistairLR112]

My incomplete attempt at the Ollama Integration to complement @sachinsachdeva
#126

Referring to this:
#146

[CLAassistant]

All committers have signed the CLA.

[anakin87]

Hello, @AlistairLR112 and @sachinsachdeva...
This is a nice attempt!

I also created an issue in this repo (#146) that reminds us what we should do to declare the integration done.
(Some actions are on our end).
Let's iterate on your draft...

• (please sign the CLA agreement)
• please add the LICENSE file to the root folder of the project
• let's remove integrations/ollama/init.py
• to make the tests run, you should create a workflow here. You can take the Elasticsearch workflow as an example.
• I will also leave some comments in the code...

[sachinsachdeva]

Hey @anakin87 ,

• I have signed the CLA (actually a few times already ) , but the status is not getting reflected for some reason, are you aware of how to trouble shoot that ?
• @AlistairLR112 has done great work so far and we should be able to wrap this up soon 💪
• As for the scope wanted to bring up potential support for streaming response and auto download Ollama model if its not available on the machine, I would suggest to complete this PR without the 2 additional features and do them separately (assuming they are voted in 😏 )

[AlistairLR112]

Hey @anakin87 ,

* I have  signed  the CLA (actually a few times already ) , but the status is not getting reflected for some  reason, are you aware of how to trouble shoot that ?

* @AlistairLR112 has done great work so far and we should be able to wrap this up soon  💪

* As for the scope wanted to bring up potential support for **streaming response** and **auto download Ollama model** if its not available on the machine, I would suggest to complete this PR without the 2 additional features and do them separately (assuming they are voted in  😏 )

Thanks @sachinsachdeva!

I agree with you about adding streaming later!

[anakin87]

@AlistairLR112 I finally managed to run Ollama for tests, taking a cue from your suggestion!
Tomorrow I will look deeply at this PR, which already seems to be in a good state...

[anakin87]

@AlistairLR112 I left some comments about aspects that can be improved.
Please let me know if you can address them...

I will also push some minor changes to this PR

• testing workflow: I will probably change something on how we spin up the Ollama service and then update the README accordingly
• add this integration to the inventory in the general README of core integrations

[AlistairLR112]

Apologies, forgot to modify a test there..

[anakin87]

@AlistairLR112 I intend to make some final refinements to docstring and other small things,
then I will merge your PR.
Thanks again!

[AlistairLR112]

@AlistairLR112 I intend to make some final refinements to docstring and other small things, then I will merge your PR. Thanks again!

I suspect some of the changes will be getting rid of my silly test cases!

Do let me know when it's all done, would like to make a post about it on linkedin if you are okay with that!

[anakin87]

I will try to merge this PR today...
Before announcing this new feature on social media, I would wait until all the steps on #146 are done (including examples and docs) so that people can start immediately using it. Once this first (big) step is merged, we can collaborate/coordinate on the other steps to speed things up.

Of course, the contribution is yours, so you can also announce it in advance if you prefer.
(BTW, it is a pleasure to collaborate with you)

[AlistairLR112]

I will try to merge this PR today... Before announcing this new feature on social media, I would wait until all the steps on #146 are done (including examples and docs) so that people can start immediately using it. Once this first (big) step is merged, we can collaborate/coordinate on the other steps to speed things up.

Of course, the contribution is yours, so you can also announce it in advance if you prefer. (BTW, it is a pleasure to collaborate with you)

Hey @anakin87, I will wait until everything is done, let me know where I can help. Has been fantastic to collab with you too! Happy to add more integrations :)

[anakin87]

This first implementation is good to go!

Merging...