feat: automated multi-platform release binaries via GitHub Actions#22
Closed
feat: automated multi-platform release binaries via GitHub Actions#22
Conversation
…ll tools Co-authored-by: lmangani <1423657+lmangani@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add multi-step GitHub action for compiling binaries
feat: automated multi-platform release binaries via GitHub Actions
Mar 11, 2026
Co-authored-by: lmangani <1423657+lmangani@users.noreply.github.com>
Updated the description for the release_tag input and commented out the windows-x64-cpu job.
lmangani
pushed a commit
that referenced
this pull request
Mar 13, 2026
…ading (-DGGML_BACKEND_DL=ON) (#22) * Fix Windows MSVC build for GGML DL mode and update to Registry API * Update src/backend.h Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Apply n_threads configuration to tokenizer backend * Fix thread params in fallback path for tokenizer * Standardize CPU backend initialization with thread params * Remove outdated comment and finalize CPU initialization * Finalize CPU initialization logic with thread param propagation * Update CMakeLists.txt --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Pascal <admin@serveurperso.com>
lmangani
pushed a commit
that referenced
this pull request
Mar 13, 2026
PR #22 duplicated the Registry API init logic in ace-understand.cpp (12 lines of boilerplate identical to backend_init). Extract a shared cpu_backend_new() helper in backend.h and replace both call sites: - backend_init(): lambda replaced by cpu_backend_new() - ace-understand.cpp: inline init replaced by cpu_backend_new() - remove unused <thread> include from ace-understand.cpp All 6 GPU/CPU modules go through backend_init() -> cpu_backend_new(). The FSQ tokenizer (CPU-only, no scheduler needed) calls cpu_backend_new() directly. (Single source of truth)
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
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.
Adds a
release.ymlworkflow that builds and attaches portable binaries for all supported backends across Linux, macOS (arm64/Metal only), and Windows on every GitHub Release publish, with aworkflow_dispatchfallback to attach to an existing release tag.Workflow matrix
linux-x64-cpu-blas-DGGML_BLAS=ONlinux-x64-cuda-DGGML_CUDA=ON(installscuda-toolkit-12-8)linux-x64-vulkan-DGGML_VULKAN=ONmacos-arm64-metalwindows-x64-cpuwindows-x64-cuda-DGGML_CUDA=ON(pre-installed)windows-x64-vulkan-DGGML_VULKAN=ON(Chocolatey SDK)Intel macOS (
macos-13) dropped — Metal/Apple Silicon only.Per-job steps
--help) → package →gh release upload --clobber.tar.gz(Linux/macOS),.zip(Windows)permissions: contents: writescoped at workflow levelci-build.ymltestsSmoke test for all 6 tools (
ace-qwen3,dit-vae,ace-understand,neural-codec,quantize,mp3-codec)