Skip to content
This repository was archived by the owner on Feb 11, 2026. It is now read-only.

Instructions to get ollama and merlinite working#198

Closed
jjasghar wants to merge 3 commits into
mainfrom
jjasghar/ollama_merlinite
Closed

Instructions to get ollama and merlinite working#198
jjasghar wants to merge 3 commits into
mainfrom
jjasghar/ollama_merlinite

Conversation

@jjasghar

Copy link
Copy Markdown
Contributor

A howto get ollama to host merlinite from huggingface

A howto get ollama to host merlinite from huggingface

Signed-off-by: JJ Asghar <awesome@ibm.com>
@jjasghar
jjasghar requested a review from a team April 22, 2024 22:56
@lhawthorn
lhawthorn requested a review from kelbrown20 April 23, 2024 11:56
@lhawthorn

Copy link
Copy Markdown
Contributor

@kelbrown20 Here is yet more documentation and I do not want to add unnecessarily to the sprawl, though @jjasghar's how to guide is valuable for new users. Do you have feedback on where we might either a) add this detail to existing docs or b) where we should link to this so as to make sure folks have the benefit of this knowledge.

@sroecker

Copy link
Copy Markdown

A howto get ollama to host merlinite from huggingface

You can also just ollama run sroecker/merlinite
I converted the model from HF and used the provided prompt template, which differs from the example here.

@jjasghar

jjasghar commented Apr 23, 2024

Copy link
Copy Markdown
Contributor Author

You can also just ollama run sroecker/merlinite

Neat! Yeah I figured someone would want to build it locally so they know exactly what they are getting. There's a chance you (or someone else with another fork) may have edited the Modelfile and made the SYSTEM look like this...

SYSTEM """
You are Mario from Super Mario Bros. Answer as Mario, the assistant, only.
"""

@sroecker

sroecker commented Apr 23, 2024 via email

Copy link
Copy Markdown

@jjasghar

Copy link
Copy Markdown
Contributor Author

Oh, I'm not saying you did. but these instructions are designed for you to pull directly from the upstream, and not rely on another name space, and know exactly what you're getting.

@sroecker

sroecker commented Apr 23, 2024 via email

Copy link
Copy Markdown

Per the comments the Prompt should be closer to the https://huggingface.co/instructlab/merlinite-7b-lab website.

Signed-off-by: JJ Asghar <awesome@ibm.com>
Forgot the "im_start"

Signed-off-by: JJ Asghar <awesome@ibm.com>
@russellb

Copy link
Copy Markdown
Contributor

I think this should at least go in a sub-directory of random misc helpful topical guides. It doesn't seem like a top-level document.

@kelbrown20

Copy link
Copy Markdown
Contributor

I think this should at least go in a sub-directory of random misc helpful topical guides. It doesn't seem like a top-level document.

I think it still works in this repo, but I agree with @russellb that it should maybe go in the docs directory rather then in the top level

@hickeyma hickeyma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jjasghar for adding this doc. It will be useful for users.

2 things I suggest:

  • I don't think naming of file in capitals is needed. Can it go lower case?
  • As per @russellb in #198 (comment), can we move it to /docs/mics directory?

@github-actions

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had activity within 90 days. It will be automatically closed if no further activity occurs within 30 days.

@github-actions github-actions Bot added the stale label Aug 30, 2024
@obuzek

obuzek commented Sep 3, 2024

Copy link
Copy Markdown
Contributor

I wonder if it makes more sense to frame this as "running arbitrary models locally with InstructLab + ollama" instead of "running Merlinite"? With more explanation on e.g. how the Modelfile is developed?

Also I think to make it cohesive with the InstructLab project, a line at the very end explaining the --endpoint-url that allows you to hit the ollama model as part of your generate command would be helpful.

@github-actions github-actions Bot removed the stale label Sep 4, 2024
@jjasghar

jjasghar commented Sep 4, 2024

Copy link
Copy Markdown
Contributor Author

I had completely forgotten about this, and it's no longer accurate. Not to mention it's not Granite, which is what our defaults should be going forward. I'm going to close this.

@jjasghar jjasghar closed this Sep 4, 2024
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.

7 participants