Llama code/code review agents - #825
Closed
aidando73 wants to merge 1 commit into
Closed
Conversation
aidando73
commented
Dec 22, 2024
| CODE_REVIEW_CYCLES = 5 | ||
|
|
||
| # Works: | ||
| MODEL_ID = "meta-llama/Llama-3.3-70B-Instruct" |
Contributor
Author
There was a problem hiding this comment.
Note that 3.3-70B for fireworks is not yet released - but will be in the next release: ogx-ai/ogx#654
aidando73
force-pushed
the
aidand-llama-code-review-demo
branch
from
December 22, 2024 07:55
e91f058 to
08e30ee
Compare
aidando73
commented
Dec 22, 2024
| ### Running the demo | ||
|
|
||
| We'll be using the fireworks llama-stack distribution to run this example - but you can use most other llama-stack distributions (instructions [here](https://llama-stack.readthedocs.io/en/latest/distributions/index.html)). | ||
| (Though note that not all distributions support structured outputs yet e.g., Ollama). |
Contributor
Author
There was a problem hiding this comment.
PR incoming for Ollama though: ogx-ai/ogx#680
Contributor
|
Thanks @aidando73 for the PR! we are reviewing it. Also, sent you a message on LD to see how we can extend the work. Feels it might have a place in llama-stack app as well. |
Contributor
Author
|
sgtm @HamidShojanazeri I actually think we shouldn't worry about this PR tbh (we can close it). This one ogx-ai/llama-stack-apps#150 👈 imo has better potential (we can measure how well it performs on SWE bench and has real world application). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Thought this would be an interesting example to have - to showcase structured outputs, tool calls and looping with Llama.
Here's a demo:
output-2024-12-22-compressed.mp4
Lmk what you think @mreso @wukaixingxp - do you think there's value in having this demo here?
Feature/Issue validation/testing
See README.md
Before submitting
Pull Request section?
to it if that's the case.
Thanks for contributing 🎉!