Skip to content

feat: add training and configuration scripts for new run#13

Merged
Ki-Seki merged 3 commits intomainfrom
feat/new-run
Sep 19, 2025
Merged

feat: add training and configuration scripts for new run#13
Ki-Seki merged 3 commits intomainfrom
feat/new-run

Conversation

@Ki-Seki
Copy link
Member

@Ki-Seki Ki-Seki commented Sep 19, 2025

No description provided.

@Ki-Seki Ki-Seki requested a review from Copilot September 19, 2025 06:31
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds training and configuration scripts for a new supervised fine-tuning (SFT) run targeting the "gim-sft-2" experiment. The implementation sets up model training using the Unsloth framework with Qwen3-4B-Instruct as the base model.

  • Implements complete training pipeline with model loading, dataset preparation, and training configuration
  • Adds configuration management for experiment parameters including batch sizes, learning rates, and data splits
  • Excludes the runs directory from mypy type checking

Reviewed Changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated 2 comments.

File Description
runs/09191-gim-sft-2/train.py Main training script implementing the complete ML pipeline from model loading to inference testing
runs/09191-gim-sft-2/configs.py Configuration module defining experiment parameters and directory structures
.pre-commit-config.yaml Excludes runs directory from mypy type checking

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Ki-Seki Ki-Seki merged commit 2e2f2b6 into main Sep 19, 2025
1 check passed
@Ki-Seki Ki-Seki deleted the feat/new-run branch September 19, 2025 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants