llm

llm copied to clipboard

This PR adds a GitHub action to run build, test, and lint (Clippy) the code on every PR.

Doesn't the build require LLVM? At least pre https://github.com/setzer22/llama-rs/pull/28 being merged I guess. Or at least that was my impression on trying it on windows a few hours ago.

You're right - I keep getting got by that 😅

Let's see what @setzer22 thinks.

You're right - I keep getting got by that sweat_smile

Let's see what @setzer22 thinks.

Yup, now that the bindgen dependency is gone, I think this CI workflow (plus my suggested change) for cargo fmt should work now. Is there a good way to tests this before merging?

You can see it running on @odysa's branch: https://github.com/odysa/llama-rs/actions/runs/4443013198/jobs/7799855788

Looks like it's failing due to #1. Can you pull in the latest ggml.c/h?

Looks like a linking error happened on Ubuntu 22.04. 👀 https://github.com/odysa/llama-rs/actions/runs/4449174539

Looks like a linking error happened on Ubuntu 22.04. 👀 https://github.com/odysa/llama-rs/actions/runs/4449174539

Yeah, that's #1. @setzer22's working on fixing that by updating to the latest ggml once they're free 🙏

Made a few changes, everything seems to be working now :smile: Thanks a lot @odysa!