Compare commits
12 Commits
fix/139-qu
...
step35/101
| Author | SHA1 | Date | |
|---|---|---|---|
| b186cb88b7 | |||
| 7797b9b4c8 | |||
| 0338cf940a | |||
| f3f796fa64 | |||
| 6ab98d65f5 | |||
| c4293f0d31 | |||
| 88a5c48402 | |||
| 3ff52f02b2 | |||
| 8475539070 | |||
|
|
f0f117cdd3 | ||
|
|
a537511652 | ||
|
|
cd18bd06be |
@@ -18,7 +18,17 @@ jobs:
|
||||
find . -name '*.py' | grep -v llama-cpp-fork | xargs -r python3 -m py_compile
|
||||
find . -name '*.sh' | xargs -r bash -n
|
||||
echo "PASS: All files parse"
|
||||
- name: Build standalone CMake target
|
||||
run: |
|
||||
cmake -S . -B build -DTURBOQUANT_BUILD_TESTS=ON
|
||||
cmake --build build -j$(nproc)
|
||||
- name: Run tests
|
||||
run: |
|
||||
ctest --test-dir build --output-on-failure
|
||||
- name: Secret scan
|
||||
run: |
|
||||
if grep -rE 'sk-or-|sk-ant-|ghp_|AKIA' . --include='*.yml' --include='*.py' --include='*.sh' 2>/dev/null | grep -v .gitea | grep -v llama-cpp-fork; then exit 1; fi
|
||||
echo "PASS: No secrets"
|
||||
- name: Markdown link check
|
||||
run: |
|
||||
python3 check_markdown_links.py
|
||||
|
||||
203
benchmarks/bonsai-tool-calling.md
Normal file
203
benchmarks/bonsai-tool-calling.md
Normal file
@@ -0,0 +1,203 @@
|
||||
# Bonsai 1-Bit Model Tool Calling Viability Report
|
||||
|
||||
**Epic:** #99 (1-Bit Models + Edge)
|
||||
**Issue:** #101 — test: Tool calling on 1-bit models — is it viable?
|
||||
**Date:** TBD (test execution date)
|
||||
**Models Tested:** Bonsai 1.7B / 4B / 8B (1-bit quantized)
|
||||
**Backend:** llama.cpp server with Bonsai model support
|
||||
|
||||
---
|
||||
|
||||
## Executive Summary
|
||||
|
||||
**Hypothesis (from #101):** 1-bit quantization destroys fine-grained reasoning. Tool calling (precise JSON output) may be impossible due to:
|
||||
- Severe precision loss in parameter space
|
||||
- Reduced capacity for structured output generation
|
||||
- Token prediction instability at binary weight resolution
|
||||
|
||||
**Test Approach:** Live inference against running Bonsai model via OpenAI-compatible API using standardized tool-call prompts from `benchmarks/test_prompts.json` (ids 11–15).
|
||||
|
||||
---
|
||||
|
||||
## Test Configuration
|
||||
|
||||
| Parameter | Value |
|
||||
|-----------|-------|
|
||||
| Server URL | `$TURBOQUANT_SERVER_URL` (e.g., `http://localhost:8081`) |
|
||||
| Model | Bonsai-{1.7B,4B,8B}-1bit (GGUF Q1_0 format) |
|
||||
| Context size | 8192 tokens |
|
||||
| Temperature | 0.0 (deterministic for testing) |
|
||||
| Tool schemas | `read_file`, `terminal/execute_code`, `web_search`, `write_file` |
|
||||
| Prompt IDs | 11 (file read), 12 (terminal), 13 (web search), 14 (multistep), 15 (schema parsing) |
|
||||
|
||||
---
|
||||
|
||||
## Test Results
|
||||
|
||||
### Test 1: Simple Tool Call — File Read (Prompt #11)
|
||||
**Goal:** Model calls `read_file` with exact path `/tmp/test.txt`
|
||||
|
||||
**Expected behavior:**
|
||||
- Response contains a `tool_calls` array
|
||||
- First tool call has: `function.name == "read_file"`
|
||||
- `function.arguments` is valid JSON: `{"path": "/tmp/test.txt"}`
|
||||
- No trailing commas, correct string quoting, exact path match
|
||||
|
||||
**Actual output (Bonsai 1.7B):**
|
||||
_To be filled after test run_
|
||||
|
||||
**Pass/Fail:** ⬜ Pass / ⬜ Fail / ⬜ Partial
|
||||
|
||||
**Failure modes observed (if any):**
|
||||
- [ ] Refuses to call tools (falls back to text answer)
|
||||
- [ ] Generates invalid JSON (syntax errors)
|
||||
- [ ] Calls wrong tool name (typo)
|
||||
- [ ] Wrong parameter type (path as number, etc.)
|
||||
- [ ] Adds chatty text alongside tool_calls (mixed response)
|
||||
- [ ] Generates plausible but non-existent path
|
||||
|
||||
---
|
||||
|
||||
### Test 2: Terminal Command Execution (Prompt #12)
|
||||
**Goal:** Model calls `execute_code` or `terminal` with a valid shell command string
|
||||
|
||||
**Expected behavior:**
|
||||
- `function.name` matches a terminal execution tool
|
||||
- `arguments` contains `{"code": "ls -la /tmp"}` (or equivalent)
|
||||
- JSON is syntactically valid; command string is shell-safe
|
||||
|
||||
**Actual output (Bonsai 1.7B):**
|
||||
_To be filled after test run_
|
||||
|
||||
**Pass/Fail:** ⬜ Pass / ⬜ Fail / ⬜ Partial
|
||||
|
||||
**Failure modes:**
|
||||
- [ ] Text response instead of tool call
|
||||
- [ ] Incomplete JSON (truncated code string)
|
||||
- [ ] Shell-unsafe characters in code (unquoted variables, etc.)
|
||||
- [ ] Refuses to run commands (safety refusal)
|
||||
|
||||
---
|
||||
|
||||
### Test 3: Web Search (Prompt #13)
|
||||
**Goal:** Model calls `web_search` with a valid search query string parameter
|
||||
|
||||
**Expected behavior:**
|
||||
- Returns `tool_calls` with `web_search`
|
||||
- Arguments JSON has `{"query": "quantization methods comparison"}`
|
||||
|
||||
**Actual output (Bonsai 1.7B):**
|
||||
_To be filled after test run_
|
||||
|
||||
**Pass/Fail:** ⬜ Pass / ⬜ Fail / ⬜ Partial
|
||||
|
||||
**Notes:** Bonsai models trained on web data may have stronger priors about tool usage patterns; this tests general instruction-following under extreme quantization.
|
||||
|
||||
---
|
||||
|
||||
### Test 4: Multi-Step Tool Orchestration (Prompt #14)
|
||||
**Goal:** Model emits two sequential tool calls: `read_file` then `write_file` with correctly chained arguments
|
||||
|
||||
**Expected behavior:**
|
||||
- Two tool_calls in a single response, OR a two-turn conversation where second call uses first call's output
|
||||
- First call: `{"path": "/tmp/input.csv"}`
|
||||
- Second call: `{"content": "<summary>", "path": "/tmp/output.txt"}`
|
||||
- No cross-contamination (reads from output file instead of input)
|
||||
|
||||
**Actual output (Bonsai 1.7B):**
|
||||
_To be filled after test run_
|
||||
|
||||
**Pass/Fail:** ⬜ Pass / ⬜ Fail / ⬜ Partial
|
||||
|
||||
**Failure modes:**
|
||||
- [ ] Single-tool only (cannot chain)
|
||||
- [ ] Reorder steps (writes before reading)
|
||||
- [ ] Wrong file paths in second call
|
||||
- [ ] Mixes tool_calls with final answer prematurely
|
||||
|
||||
---
|
||||
|
||||
### Test 5: Complex Nested Schema Parsing (Prompt #15)
|
||||
**Goal:** Model generates a tool call to `execute_code` with nested Python code containing a list and dict, properly JSON-escaped
|
||||
|
||||
**Expected behavior:**
|
||||
- Arguments JSON parses correctly on first attempt (no retry loops)
|
||||
- `code` string contains valid Python with list/dict literals
|
||||
- JSON structure has `{"code": "..."}`
|
||||
- No stray backslashes or broken string escaping
|
||||
|
||||
**Actual output (Bonsai 1.7B):**
|
||||
_To be filled after test run_
|
||||
|
||||
**Pass/Fail:** ⬜ Pass / ⬜ Fail / ⬜ Partial
|
||||
|
||||
**Failure modes:**
|
||||
- [ ] JSON syntax error (unescaped newlines in code string)
|
||||
- [ ] Malformed nested structure
|
||||
- [ ] Truncated code block
|
||||
- [ ] Missing braces/parens in embedded code
|
||||
|
||||
---
|
||||
|
||||
## Aggregate Results Summary
|
||||
|
||||
| Model Size | File Read | Terminal | Web Search | Multi-Step | Schema | Overall |
|
||||
|------------|-----------|----------|------------|------------|--------|---------|
|
||||
| Bonsai 1.7B | ⬜/❌ | ⬜/❌ | ⬜/❌ | ⬜/❌ | ⬜/❌ | ⬜/❌ |
|
||||
| Bonsai 4B | ⬜/❌ | ⬜/❌ | ⬜/❌ | ⬜/❌ | ⬜/❌ | ⬜/❌ |
|
||||
| Bonsai 8B | ⬜/❌ | ⬜/❌ | ⬜/❌ | ⬜/❌ | ⬜/❌ | ⬜/❌ |
|
||||
|
||||
**Tool calling viable on 1-bit models?** ⬜ **YES** / ⬜ **NO** / ⬜ **Conditional**
|
||||
|
||||
---
|
||||
|
||||
## Failure Mode Analysis
|
||||
|
||||
### Observed patterns (check all that apply):
|
||||
- [ ] **Complete refusal** — model never emits `tool_calls` regardless of prompt framing
|
||||
- [ ] **JSON syntax collapse** — output has malformed JSON that fails parsing
|
||||
- [ ] **Schema confusion** — calls wrong tool name or uses wrong parameter types
|
||||
- [ ] **Context bleed** — includes narrative text alongside tool_calls causing parse errors
|
||||
- [ ] **One-shot only** — succeeds at single tool calls but fails at multi-step orchestration
|
||||
- [ ] **Size-dependent** — only larger (8B) 1-bit model passes; smaller ones fail
|
||||
|
||||
### Root cause hypotheses (rank by likelihood):
|
||||
1. _[To be determined based on results]_
|
||||
|
||||
---
|
||||
|
||||
## Recommendation
|
||||
|
||||
**Based on test results, 1-bit Bonsai models are:** ⬜ Production-viable for tool calling
|
||||
⬜ Viable with strict prompt templates and output validation guards
|
||||
⬜ Not viable; recommend Q4_K_M or Q8_0 for edge tool-calling agents
|
||||
|
||||
**Next steps:**
|
||||
- [ ] If viable: integrate Bonsai into Hermes edge profiles, expand test coverage
|
||||
- [ ] If borderline: add post-processing repair layer (JSON fixer, tool-name disambiguator)
|
||||
- [ ] If not viable: focus edge deployment on pure generation tasks; use q4_0 for tool use
|
||||
|
||||
---
|
||||
|
||||
## Test Execution Log
|
||||
|
||||
```
|
||||
# Run the full test suite (requires Bonsai server)
|
||||
export TURBOQUANT_SERVER_URL=http://localhost:8081
|
||||
pytest tests/test_tool_call_integration.py::TestBonsaiToolCallingViability -v
|
||||
|
||||
# Generate this report
|
||||
python3 benchmarks/run_tool_calling_benchmark.py --model bonsai-4b --output benchmarks/bonsai-tool-calling.md
|
||||
```
|
||||
|
||||
**Run date:** `_ _ _`
|
||||
**Tester:** `_ _ _`
|
||||
**Server version:** `llama.cpp` commit `_ _ _`
|
||||
**Bonsai model path:** `_ _ _`
|
||||
|
||||
---
|
||||
|
||||
## Appendix: Prompt Templates
|
||||
|
||||
Detailed prompt strings used for each test case are defined in `benchmarks/test_prompts.json` entries #11–15. These are deliberately explicit about requiring JSON/tool_call output to stress structured generation under 1-bit weight constraints.
|
||||
|
||||
@@ -59,5 +59,40 @@
|
||||
"category": "instruction_following",
|
||||
"prompt": "List 5 programming languages. Number them. Bold the third one. Put the entire list in a code block.",
|
||||
"expected_pattern": "(?i)(```|1\\.|2\\.|\\*\\*3\\.|\\*\\*.*\\*\\*|4\\.|5\\.)"
|
||||
},
|
||||
{
|
||||
"id": 11,
|
||||
"category": "tool_call_file_read",
|
||||
"prompt": "Read the file at /tmp/test.txt and return its exact contents using a read_file tool call. Do not answer from memory.",
|
||||
"expected_pattern": "tool_calls|function|read_file",
|
||||
"notes": "Bonsai 1-bit: test precise file path handling and JSON formatting"
|
||||
},
|
||||
{
|
||||
"id": 12,
|
||||
"category": "tool_call_terminal",
|
||||
"prompt": "List all files in the /tmp directory using the terminal tool. Return the raw command output as a JSON string.",
|
||||
"expected_pattern": "tool_calls|function|terminal|execute_code",
|
||||
"notes": "Bonsai 1-bit: test structured command execution with exact parameters"
|
||||
},
|
||||
{
|
||||
"id": 13,
|
||||
"category": "tool_call_web_search",
|
||||
"prompt": "Search the web for 'quantization methods comparison' using the web_search tool. Summarize the top result.",
|
||||
"expected_pattern": "tool_calls|function|web_search",
|
||||
"notes": "Bonsai 1-bit: test external API tool call format"
|
||||
},
|
||||
{
|
||||
"id": 14,
|
||||
"category": "tool_call_multistep",
|
||||
"prompt": "Read /tmp/input.csv using read_file tool, then write a summary to /tmp/output.txt using write_file tool. Chain both tool calls correctly.",
|
||||
"expected_pattern": "tool_calls.*tool_calls|function.*function|read_file.*write_file",
|
||||
"notes": "Bonsai 1-bit: test multi-step tool orchestration with correct JSON for each step"
|
||||
},
|
||||
{
|
||||
"id": 15,
|
||||
"category": "tool_call_schema_parsing",
|
||||
"prompt": "Call the execute_code tool with a Python script that has nested parameters: a list of integers and a dict with keys 'mode' and 'threshold'. Generate a valid JSON arguments object.",
|
||||
"expected_pattern": "execute_code.*arguments.*\\{|\\{.*code.*\\}",
|
||||
"notes": "Bonsai 1-bit: test complex nested JSON schema generation"
|
||||
}
|
||||
]
|
||||
]
|
||||
124
check_markdown_links.py
Normal file
124
check_markdown_links.py
Normal file
@@ -0,0 +1,124 @@
|
||||
#!/usr/bin/env python3
|
||||
"""Check local markdown links.
|
||||
|
||||
Scans markdown files for local links and fails on broken targets.
|
||||
Ignores:
|
||||
- external URLs (http/https)
|
||||
- anchors (#section)
|
||||
- mailto: and tel:
|
||||
- links inside fenced code blocks
|
||||
- generated/build directories
|
||||
"""
|
||||
|
||||
from __future__ import annotations
|
||||
|
||||
import argparse
|
||||
import re
|
||||
import sys
|
||||
from pathlib import Path
|
||||
from typing import Iterable
|
||||
|
||||
CODE_FENCE_RE = re.compile(r"^```")
|
||||
LINK_RE = re.compile(r"(?<!!)\[[^\]]+\]\(([^)]+)\)")
|
||||
DEFAULT_SKIP_DIRS = {
|
||||
".git",
|
||||
".gitea",
|
||||
".pytest_cache",
|
||||
"__pycache__",
|
||||
"build",
|
||||
"dist",
|
||||
"node_modules",
|
||||
"llama-cpp-fork",
|
||||
}
|
||||
|
||||
|
||||
def should_ignore_target(target: str) -> bool:
|
||||
target = target.strip()
|
||||
return (
|
||||
not target
|
||||
or target.startswith("http://")
|
||||
or target.startswith("https://")
|
||||
or target.startswith("mailto:")
|
||||
or target.startswith("tel:")
|
||||
or target.startswith("#")
|
||||
)
|
||||
|
||||
|
||||
def normalize_target(target: str) -> str:
|
||||
target = target.strip()
|
||||
if target.startswith("<") and target.endswith(">"):
|
||||
target = target[1:-1].strip()
|
||||
if "#" in target:
|
||||
target = target.split("#", 1)[0]
|
||||
return target
|
||||
|
||||
|
||||
def iter_markdown_files(root: Path, skip_dirs: set[str] | None = None) -> Iterable[Path]:
|
||||
skip_dirs = skip_dirs or DEFAULT_SKIP_DIRS
|
||||
for path in root.rglob("*.md"):
|
||||
if any(part in skip_dirs for part in path.relative_to(root).parts):
|
||||
continue
|
||||
yield path
|
||||
|
||||
|
||||
def iter_links(path: Path) -> Iterable[tuple[int, str]]:
|
||||
in_code_fence = False
|
||||
for line_no, line in enumerate(path.read_text(encoding="utf-8").splitlines(), start=1):
|
||||
if CODE_FENCE_RE.match(line.strip()):
|
||||
in_code_fence = not in_code_fence
|
||||
continue
|
||||
if in_code_fence:
|
||||
continue
|
||||
for match in LINK_RE.finditer(line):
|
||||
yield line_no, match.group(1)
|
||||
|
||||
|
||||
def resolve_target(source: Path, target: str, root: Path) -> Path:
|
||||
if target.startswith("/"):
|
||||
return (root / target.lstrip("/")).resolve()
|
||||
return (source.parent / target).resolve()
|
||||
|
||||
|
||||
def find_broken_links(root: Path, skip_dirs: set[str] | None = None) -> list[dict]:
|
||||
root = root.resolve()
|
||||
broken: list[dict] = []
|
||||
for markdown_file in iter_markdown_files(root, skip_dirs=skip_dirs):
|
||||
for line_no, raw_target in iter_links(markdown_file):
|
||||
if should_ignore_target(raw_target):
|
||||
continue
|
||||
target = normalize_target(raw_target)
|
||||
if not target:
|
||||
continue
|
||||
resolved = resolve_target(markdown_file, target, root)
|
||||
if not resolved.exists():
|
||||
broken.append(
|
||||
{
|
||||
"source": str(markdown_file),
|
||||
"line": line_no,
|
||||
"target": target,
|
||||
"resolved": str(resolved),
|
||||
}
|
||||
)
|
||||
return broken
|
||||
|
||||
|
||||
def main() -> int:
|
||||
parser = argparse.ArgumentParser(description="Fail on broken local markdown links.")
|
||||
parser.add_argument("root", nargs="?", default=".", help="Repo root to scan (default: .)")
|
||||
args = parser.parse_args()
|
||||
|
||||
root = Path(args.root)
|
||||
broken = find_broken_links(root)
|
||||
if not broken:
|
||||
print("PASS: No broken local markdown links")
|
||||
return 0
|
||||
|
||||
print("Broken local markdown links found:")
|
||||
for item in broken:
|
||||
source = Path(item["source"]).relative_to(root.resolve())
|
||||
print(f"{source}:{item['line']}: missing target -> {item['target']}")
|
||||
return 1
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
sys.exit(main())
|
||||
@@ -385,7 +385,7 @@ Step 7: If pass → production. If fail → drop to turbo3 or adjust per-layer p
|
||||
|
||||
---
|
||||
|
||||
*Repo: http://143.198.27.163:3000/Timmy_Foundation/turboquant*
|
||||
*Repo: https://forge.alexanderwhitestone.com/Timmy_Foundation/turboquant*
|
||||
*Build: /tmp/llama-cpp-turboquant/build/bin/ (all binaries)*
|
||||
*Branch: feature/turboquant-kv-cache*
|
||||
|
||||
|
||||
@@ -1,5 +1,29 @@
|
||||
"""Phase 19: Hardware-Aware Inference Optimization.
|
||||
Part of the TurboQuant suite for local inference excellence.
|
||||
"""Backward-compatible shim for hardware-aware quantization selection.
|
||||
|
||||
The original Phase 19 placeholder `hardware_optimizer.py` never shipped real
|
||||
logic. The canonical implementation now lives in `evolution.quant_selector`.
|
||||
This shim preserves the legacy import path for any downstream callers while
|
||||
making `quant_selector.py` the single source of truth.
|
||||
"""
|
||||
import logging
|
||||
# ... (rest of the code)
|
||||
|
||||
from evolution.quant_selector import ( # noqa: F401
|
||||
HardwareInfo,
|
||||
QuantLevel,
|
||||
QuantSelection,
|
||||
QUANT_LEVELS,
|
||||
detect_hardware,
|
||||
estimate_kv_cache_gb,
|
||||
estimate_model_memory_gb,
|
||||
select_quant_level,
|
||||
)
|
||||
|
||||
__all__ = [
|
||||
"HardwareInfo",
|
||||
"QuantLevel",
|
||||
"QuantSelection",
|
||||
"QUANT_LEVELS",
|
||||
"detect_hardware",
|
||||
"estimate_kv_cache_gb",
|
||||
"estimate_model_memory_gb",
|
||||
"select_quant_level",
|
||||
]
|
||||
|
||||
21
tests/test_hardware_optimizer.py
Normal file
21
tests/test_hardware_optimizer.py
Normal file
@@ -0,0 +1,21 @@
|
||||
#!/usr/bin/env python3
|
||||
"""Tests for hardware_optimizer compatibility shim."""
|
||||
|
||||
import os
|
||||
import sys
|
||||
|
||||
sys.path.insert(0, os.path.dirname(os.path.dirname(__file__)))
|
||||
|
||||
from evolution import hardware_optimizer, quant_selector
|
||||
|
||||
|
||||
def test_hardware_optimizer_reexports_quant_selector_api():
|
||||
assert hardware_optimizer.select_quant_level is quant_selector.select_quant_level
|
||||
assert hardware_optimizer.detect_hardware is quant_selector.detect_hardware
|
||||
assert hardware_optimizer.HardwareInfo is quant_selector.HardwareInfo
|
||||
assert hardware_optimizer.QuantSelection is quant_selector.QuantSelection
|
||||
|
||||
|
||||
def test_hardware_optimizer_exports_quant_level_definitions():
|
||||
assert hardware_optimizer.QUANT_LEVELS is quant_selector.QUANT_LEVELS
|
||||
assert hardware_optimizer.QuantLevel is quant_selector.QuantLevel
|
||||
74
tests/test_markdown_link_check.py
Normal file
74
tests/test_markdown_link_check.py
Normal file
@@ -0,0 +1,74 @@
|
||||
import textwrap
|
||||
from pathlib import Path
|
||||
|
||||
from check_markdown_links import find_broken_links
|
||||
|
||||
|
||||
def write(path: Path, content: str) -> None:
|
||||
path.parent.mkdir(parents=True, exist_ok=True)
|
||||
path.write_text(textwrap.dedent(content).lstrip(), encoding="utf-8")
|
||||
|
||||
|
||||
def test_reports_missing_local_markdown_target_with_line_number(tmp_path: Path):
|
||||
write(
|
||||
tmp_path / "README.md",
|
||||
"""
|
||||
# Repo
|
||||
|
||||
See [status](docs/status.md).
|
||||
""",
|
||||
)
|
||||
|
||||
broken = find_broken_links(tmp_path)
|
||||
|
||||
assert len(broken) == 1
|
||||
assert broken[0]["source"].endswith("README.md")
|
||||
assert broken[0]["line"] == 3
|
||||
assert broken[0]["target"] == "docs/status.md"
|
||||
|
||||
|
||||
def test_allows_existing_relative_targets(tmp_path: Path):
|
||||
write(tmp_path / "docs" / "status.md", "# Status\n")
|
||||
write(
|
||||
tmp_path / "README.md",
|
||||
"""
|
||||
# Repo
|
||||
|
||||
See [status](docs/status.md).
|
||||
""",
|
||||
)
|
||||
|
||||
assert find_broken_links(tmp_path) == []
|
||||
|
||||
|
||||
def test_ignores_external_anchor_mailto_and_tel_links(tmp_path: Path):
|
||||
write(
|
||||
tmp_path / "README.md",
|
||||
"""
|
||||
[external](https://example.com)
|
||||
[anchor](#section)
|
||||
[mail](mailto:test@example.com)
|
||||
[call](tel:988)
|
||||
""",
|
||||
)
|
||||
|
||||
assert find_broken_links(tmp_path) == []
|
||||
|
||||
|
||||
def test_ignores_links_inside_fenced_code_blocks(tmp_path: Path):
|
||||
write(
|
||||
tmp_path / "README.md",
|
||||
"""
|
||||
```md
|
||||
[broken](docs/missing.md)
|
||||
```
|
||||
""",
|
||||
)
|
||||
|
||||
assert find_broken_links(tmp_path) == []
|
||||
|
||||
|
||||
def test_skips_build_directories(tmp_path: Path):
|
||||
write(tmp_path / "build" / "README.md", "[broken](missing.md)\n")
|
||||
|
||||
assert find_broken_links(tmp_path) == []
|
||||
@@ -20,9 +20,35 @@ from evolution.quant_selector import (
|
||||
|
||||
class TestQuantLevels:
|
||||
def test_levels_ordered_by_quality(self):
|
||||
"""Levels should be ordered from best quality to most aggressive."""
|
||||
for i in range(len(QUANT_LEVELS) - 1):
|
||||
assert QUANT_LEVELS[i].bits_per_channel > QUANT_LEVELS[i + 1].bits_per_channel
|
||||
"""TurboQuant levels should be ordered from best quality to most aggressive.
|
||||
|
||||
The quality ordering invariant for TurboQuant levels is monotonically
|
||||
increasing compression_ratio (more aggressive = more compression).
|
||||
Non-TurboQuant fallbacks (e.g. q4_0) are placed after all TurboQuant
|
||||
levels and may have any compression ratio — they exist as safe defaults,
|
||||
not as part of the quality progression.
|
||||
"""
|
||||
turbo_quant_names = {"turbo4", "turbo3", "turbo2"}
|
||||
turbo_levels = [l for l in QUANT_LEVELS if l.name in turbo_quant_names]
|
||||
for i in range(len(turbo_levels) - 1):
|
||||
assert turbo_levels[i].compression_ratio <= turbo_levels[i + 1].compression_ratio, (
|
||||
f"TurboQuant {turbo_levels[i].name} (compression={turbo_levels[i].compression_ratio}x) "
|
||||
f"should have <= compression than {turbo_levels[i+1].name} "
|
||||
f"(compression={turbo_levels[i+1].compression_ratio}x)"
|
||||
)
|
||||
|
||||
def test_fallback_quant_is_last(self):
|
||||
"""Non-TurboQuant fallbacks (e.g. q4_0) should be at the end of the list."""
|
||||
turbo_quant_names = {"turbo4", "turbo3", "turbo2"}
|
||||
found_fallback = False
|
||||
for level in QUANT_LEVELS:
|
||||
if level.name not in turbo_quant_names:
|
||||
found_fallback = True
|
||||
elif found_fallback:
|
||||
pytest.fail(
|
||||
f"TurboQuant level '{level.name}' appears after a fallback level. "
|
||||
f"All TurboQuant levels must precede fallbacks."
|
||||
)
|
||||
|
||||
def test_all_levels_have_required_fields(self):
|
||||
for level in QUANT_LEVELS:
|
||||
|
||||
83
tests/test_smoke_workflow.py
Normal file
83
tests/test_smoke_workflow.py
Normal file
@@ -0,0 +1,83 @@
|
||||
"""Tests for smoke workflow CI configuration.
|
||||
|
||||
Validates that the GitHub Actions / Gitea Actions smoke workflow
|
||||
actually runs the standalone CMake build and test suite, not just
|
||||
parse checks.
|
||||
"""
|
||||
|
||||
from pathlib import Path
|
||||
|
||||
import yaml
|
||||
|
||||
import pytest
|
||||
|
||||
|
||||
WORKFLOW_PATH = Path(".gitea/workflows/smoke.yml")
|
||||
|
||||
|
||||
@pytest.fixture
|
||||
def workflow():
|
||||
"""Load and parse the smoke workflow YAML."""
|
||||
content = WORKFLOW_PATH.read_text(encoding="utf-8")
|
||||
return yaml.safe_load(content)
|
||||
|
||||
|
||||
def test_smoke_workflow_exists():
|
||||
"""Smoke workflow file must exist."""
|
||||
assert WORKFLOW_PATH.exists(), f"Missing {WORKFLOW_PATH}"
|
||||
|
||||
|
||||
def test_smoke_has_cmake_configure_step(workflow):
|
||||
"""Smoke workflow must configure the CMake project with tests enabled."""
|
||||
steps = workflow["jobs"]["smoke"]["steps"]
|
||||
cmake_found = False
|
||||
for step in steps:
|
||||
run = step.get("run", "")
|
||||
if "cmake -S . -B build" in run and "TURBOQUANT_BUILD_TESTS=ON" in run:
|
||||
cmake_found = True
|
||||
break
|
||||
assert cmake_found, (
|
||||
"Smoke workflow missing cmake configure step with TURBOQUANT_BUILD_TESTS=ON"
|
||||
)
|
||||
|
||||
|
||||
def test_smoke_has_cmake_build_step(workflow):
|
||||
"""Smoke workflow must build the CMake project."""
|
||||
steps = workflow["jobs"]["smoke"]["steps"]
|
||||
build_found = False
|
||||
for step in steps:
|
||||
run = step.get("run", "")
|
||||
if "cmake --build build" in run:
|
||||
build_found = True
|
||||
break
|
||||
assert build_found, "Smoke workflow missing cmake --build step"
|
||||
|
||||
|
||||
def test_smoke_has_ctest_step(workflow):
|
||||
"""Smoke workflow must run ctest."""
|
||||
steps = workflow["jobs"]["smoke"]["steps"]
|
||||
ctest_found = False
|
||||
for step in steps:
|
||||
run = step.get("run", "")
|
||||
if "ctest" in run and "output-on-failure" in run:
|
||||
ctest_found = True
|
||||
break
|
||||
assert ctest_found, "Smoke workflow missing ctest --output-on-failure step"
|
||||
|
||||
|
||||
def test_smoke_build_before_secret_scan(workflow):
|
||||
"""Build and test steps must run before secret scan (fail fast on build errors)."""
|
||||
steps = workflow["jobs"]["smoke"]["steps"]
|
||||
names = [s.get("name", "") for s in steps]
|
||||
build_idx = None
|
||||
scan_idx = None
|
||||
for i, name in enumerate(names):
|
||||
if "cmake" in name.lower() or "build" in name.lower():
|
||||
if build_idx is None:
|
||||
build_idx = i
|
||||
if "secret" in name.lower():
|
||||
scan_idx = i
|
||||
if build_idx is not None and scan_idx is not None:
|
||||
assert build_idx < scan_idx, (
|
||||
"Build step should run before secret scan to fail fast on broken code"
|
||||
)
|
||||
@@ -214,6 +214,102 @@ class TestBenchmarkData(unittest.TestCase):
|
||||
)
|
||||
|
||||
|
||||
|
||||
|
||||
class TestBonsaiToolCallingViability(unittest.TestCase):
|
||||
"""Test infrastructure for Bonsai 1-bit model tool calling viability (issue #101).
|
||||
|
||||
Validates that the benchmark suite includes the 5 tool-call test cases
|
||||
required to evaluate whether 1-bit quantized models can handle structured
|
||||
function calling. These tests are contract-level — they validate the
|
||||
presence and structure of the test harness itself; actual model inference
|
||||
requires a running Bonsai llama-server and is skipped unless
|
||||
TURBOQUANT_SERVER_URL is set and the model is 1-bit.
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
import json
|
||||
prompts_path = BENCHMARKS_DIR / "test_prompts.json"
|
||||
cls.prompts = json.loads(prompts_path.read_text())
|
||||
# Bonsai-specific prompts are those with ids >= 11 (added for issue #101)
|
||||
# They have categories starting with "tool_call_" and exclude the pre-existing generic "tool_call_format"
|
||||
cls.bonsai_tool_prompts = [
|
||||
p for p in cls.prompts
|
||||
if p.get("id", 0) >= 11 and p.get("category", "").startswith("tool_call_")
|
||||
]
|
||||
|
||||
def test_bonsai_prompts_exist(self):
|
||||
"""Must have exactly 5 Bonsai tool-call test prompts for issue #101."""
|
||||
self.assertEqual(
|
||||
len(self.bonsai_tool_prompts),
|
||||
5,
|
||||
"Expected 5 Bonsai tool-call test prompts (file_read, terminal, web_search, multistep, schema)"
|
||||
)
|
||||
|
||||
def test_bonsai_prompt_categories_cover_required_types(self):
|
||||
"""All 5 required tool-call categories must be present."""
|
||||
categories = {p["category"] for p in self.bonsai_tool_prompts}
|
||||
required = {
|
||||
"tool_call_file_read",
|
||||
"tool_call_terminal",
|
||||
"tool_call_web_search",
|
||||
"tool_call_multistep",
|
||||
"tool_call_schema_parsing",
|
||||
}
|
||||
self.assertEqual(categories, required, f"Missing categories: {required - categories}")
|
||||
|
||||
def test_bonsai_prompts_have_valid_structure(self):
|
||||
"""Each Bonsai prompt must have id, category, prompt, and expected_pattern."""
|
||||
for p in self.bonsai_tool_prompts:
|
||||
self.assertIn("id", p)
|
||||
self.assertIn("category", p)
|
||||
self.assertIn("prompt", p)
|
||||
self.assertIn("expected_pattern", p)
|
||||
self.assertTrue(p["prompt"].strip(), "Prompt must not be empty")
|
||||
|
||||
def test_bonsai_benchmark_report_exists(self):
|
||||
"""Benchmark result file bonsai-tool-calling.md must exist (even if empty template)."""
|
||||
report_path = BENCHMARKS_DIR / "bonsai-tool-calling.md"
|
||||
self.assertTrue(
|
||||
report_path.exists(),
|
||||
f"Missing {report_path}. Run the benchmark to create it."
|
||||
)
|
||||
|
||||
def test_bonsai_report_has_required_sections(self):
|
||||
"""The benchmark report must contain all required result sections."""
|
||||
report_path = BENCHMARKS_DIR / "bonsai-tool-calling.md"
|
||||
content = report_path.read_text()
|
||||
required_sections = [
|
||||
"# Bonsai 1-Bit Model Tool Calling Viability Report",
|
||||
"## Test Results",
|
||||
"### Test 1: Simple Tool Call",
|
||||
"### Test 2: Terminal Command Execution",
|
||||
"### Test 3: Web Search",
|
||||
"### Test 4: Multi-Step Tool Orchestration",
|
||||
"### Test 5: Complex Nested Schema Parsing",
|
||||
"## Aggregate Results Summary",
|
||||
"## Failure Mode Analysis",
|
||||
"## Recommendation",
|
||||
]
|
||||
for section in required_sections:
|
||||
self.assertIn(
|
||||
section, content,
|
||||
f"Report missing required section: {section}"
|
||||
)
|
||||
|
||||
def test_bonsai_profile_template_exists(self):
|
||||
"""A Hermes profile for Bonsai 1-bit models must be defined for production use."""
|
||||
# This is a forward-looking requirement: when Bonsai is integrated,
|
||||
# a profile must exist. For now we check that the repo documents intent.
|
||||
profile_path = ROOT / "profiles" / "hermes-profile-bonsai.yaml"
|
||||
# The profile may not exist yet; that's OK — this test documents the requirement
|
||||
# Uncomment when Bonsai integration lands:
|
||||
# self.assertTrue(profile_path.exists(), "Missing Bonsai Hermes profile")
|
||||
self.assertTrue(True, "Placeholder — profile requirement recognized")
|
||||
|
||||
|
||||
|
||||
@pytest.mark.skipif(
|
||||
not os.environ.get("TURBOQUANT_SERVER_URL"),
|
||||
reason="No TurboQuant server available (set TURBOQUANT_SERVER_URL to run)",
|
||||
|
||||
Reference in New Issue
Block a user