-
Notifications
You must be signed in to change notification settings - Fork 64
Update validator model #229
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@dbobrenko Unfortunately I cannot add you as a reviewer in this space but please consider yourself as a reviewer of all PRs of this repo (specially the ones with tag |
dbobrenko
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we update min_compute.yaml also, and README.md?
Overall, LGTM!
dbobrenko
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: Dmytro Bobrenko <17252809+dbobrenko@users.noreply.github.com>
Co-authored-by: Dmytro Bobrenko <17252809+dbobrenko@users.noreply.github.com>
bkb2135
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good and has been run on main stabely.
--neuron.gpusand--neuron.llm_max_allowed_memory_in_gbnew parameters and integrationcasperhansen/llama-3-70b-instruct-awq