Repository navigation
docs(examples): reorder router cost summary - #850
Conversation
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Cost table and notes examples/train-a-router/README.md |
The table combines answer-model and router costs into an estimated total-cost column. The note distinguishes Laya serving estimates from SIE Cloud embedding prices and defines savings relative to the unrounded total. |
Possibly related PRs
- superlinked/sie#848: Updates the same README comparison with embedding and Laya serving costs.
- superlinked/sie#795: Adds the README walkthrough and router cost comparison.
Suggested reviewers: dragosboca
Priority:
Merge Risk: 
Minimal · up to c81c0
-
The documentation-only change is ready to merge.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
Reorder the opening router table so answer accuracy, accuracy loss, total cost and savings appear before the Sol/Luna/Laya/embedding breakdown. Remove the approximation symbols and keep one explanation of estimated costs and excluded hosting/training costs above the table.
All ten rows retain their existing numerical values. Verified the exact column mapping, absence of approximation symbols, and a clean whitespace check. This change only edits documentation.
Summary by CodeRabbit