|
|
|
|
|
|
by BowBun
569 days ago
|
|
|
Author describes why it took as long as it did in the post, so I don't think they're trying to be disingenous. Getting minor changes merged upstream in large projects is difficult for newer concepts since you need adoption and support. Full release seems to contain more code[1], and author references the llama.cpp pre-work and that author as well This person is also not a core contributor, so this reads as a hobbyist and fan of AI dev that is writing about their work. Nothing to be ashamed of IMO. [1] - https://github.com/ollama/ollama/compare/v0.4.7...v0.4.8-rc0 |
|
|
Bingo, that's me!
I suspect the OP didn't actually read the post.
1. As you pointed out, it's about getting the feature working, enabled and contributed into Ollama, not in llama.cpp
2. Digging through git commits isn't useful when you work hard to squash commits before merging a PR, there were a _lot_ over the last 5 months.
3. While I'm not a go dev (and the introduction of cgo part way through that threw me a bit) there certainly were technicalities along the way, I suspect they not only didn't both to read the post, they also didn't bother to read the PR.
Also, just to clarify - I didn't even share this here, it's just my personal blog of things I try to remember I did when I look back at them years later.