Skip to content

fix: use rice when embedding large binaries #5309

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

Merged
merged 5 commits into from
May 4, 2025
Merged

fix: use rice when embedding large binaries #5309

merged 5 commits into from
May 4, 2025

Conversation

mudler
Copy link
Owner

@mudler mudler commented May 3, 2025

Description

This PR fixes #

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Copy link

netlify bot commented May 3, 2025

Deploy Preview for localai ready!

Name Link
🔨 Latest commit 59164fe
🔍 Latest deploy log https://app.netlify.com/sites/localai/deploys/681738c23041e8000867c04f
😎 Deploy Preview https://deploy-preview-5309--localai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mudler mudler force-pushed the feat/go-rice branch 3 times, most recently from af46c56 to c35cebd Compare May 3, 2025 17:00
@mudler mudler changed the title Feat/go rice feat: use rice when embedding large binaries May 3, 2025
@mudler mudler changed the title feat: use rice when embedding large binaries fix: use rice when embedding large binaries May 3, 2025
@mudler mudler force-pushed the feat/go-rice branch 5 times, most recently from cacad7a to 789c871 Compare May 3, 2025 17:21
@@ -56,7 +58,7 @@
// If there is a lib directory, set LD_LIBRARY_PATH to include it
// we might use this mechanism to carry over e.g. Nvidia CUDA libraries
// from the embedded FS to the target directory
library.LoadExtractedLibs(extractDir)
library.LoadExtractedLibs(backendAssetsDir)

Check warning

Code scanning / gosec

Errors unhandled Warning

Errors unhandled
Golang embed FS has a hard limit that we might exceed when providing
many binary alternatives.

Signed-off-by: Ettore Di Giacinto <[email protected]>
Signed-off-by: Ettore Di Giacinto <[email protected]>
mudler added 2 commits May 4, 2025 11:17
Signed-off-by: Ettore Di Giacinto <[email protected]>
Signed-off-by: Ettore Di Giacinto <[email protected]>
@mudler mudler marked this pull request as ready for review May 4, 2025 14:42
@mudler mudler merged commit 21bdfe5 into master May 4, 2025
25 checks passed
@mudler mudler deleted the feat/go-rice branch May 4, 2025 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant