Skip to content
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

llama.cpp: throttle #183973

Closed
wants to merge 1 commit into from
Closed

llama.cpp: throttle #183973

wants to merge 1 commit into from

Conversation

BrewTestBot
Copy link
Member

@BrewTestBot BrewTestBot commented Sep 8, 2024

Created with brew bump-formula-pr.

Edit by FX: Highjacked to propose throttling.

@github-actions github-actions bot added the bump-formula-pr PR was created using `brew bump-formula-pr` label Sep 8, 2024
@fxcoudert fxcoudert changed the title llama.cpp 3707 llama.cpp: throttle Sep 8, 2024
@fxcoudert
Copy link
Member

6th update in the last 24 hours. I am proposing a throttle for this one.

@@ -11,6 +11,7 @@ class LlamaCpp < Formula
livecheck do
url :stable
regex(/^v?b(\d+(?:\.\d+)*)$/i)
throttle 5
Copy link
Member

Choose a reason for hiding this comment

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

@bevanjkay
Copy link
Member

The throttle dsl doesn't work on this Formula at the moment, see previous attempt #173213

@fxcoudert
Copy link
Member

I've opened a brew bug at Homebrew/brew#18293

@carlocab
Copy link
Member

carlocab commented Sep 9, 2024

Closing for now since this requires a fix in brew that I don't see coming very soon.

@carlocab carlocab closed this Sep 9, 2024
@carlocab carlocab deleted the bump-llama.cpp-3707 branch September 9, 2024 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants