Skip to content

Conversation

@andyzhangx
Copy link
Collaborator

Reason for Change:

docs: fix preset_generator.py doc links due to rename from calculate_model_weight_and_bytes_per_token.py

Requirements

  • added unit tests and e2e tests (if applicable).

Issue Fixed:

Notes for Reviewers:

@kaito-pr-agent
Copy link

Title

docs: Update references to renamed preset_generator.py script


Description

  • Updated documentation references to reflect script renaming

  • Fixed broken links in model interface and SKU calculation docs

  • Replaced old script name with new preset_generator.py

  • Ensured consistency across code and documentation


Changes walkthrough 📝

Relevant files
Documentation
interface.go
Updated script reference in model interface documentation

pkg/model/interface.go

  • Replaced reference to old script name with new preset_generator.py
  • Updated documentation comment for model parameter calculation
  • +1/-1     
    model-sku-calculation.md
    Fixed script references in SKU calculation documentation 

    presets/workspace/generator/model-sku-calculation.md

  • Updated 2 occurrences of old script name to preset_generator.py
  • Corrected links in model weight size calculation section
  • Fixed Bytes_per_token calculation reference
  • +2/-2     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @kaito-pr-agent
    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @Fei-Guo Fei-Guo merged commit 5d8baa3 into kaito-project:main Dec 22, 2025
    17 of 19 checks passed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    Status: Done

    Development

    Successfully merging this pull request may close these issues.

    2 participants