Skip to content

[docs] Create data directory in MLX Phi-4 reasoning notebook#535

Open
danyalahmed1995 wants to merge 1 commit intomicrosoft:mainfrom
danyalahmed1995:docs/create-mlx-phi4-reasoning-data-dir
Open

[docs] Create data directory in MLX Phi-4 reasoning notebook#535
danyalahmed1995 wants to merge 1 commit intomicrosoft:mainfrom
danyalahmed1995:docs/create-mlx-phi4-reasoning-data-dir

Conversation

@danyalahmed1995
Copy link
Copy Markdown

Summary

Creates the ./data directory before the MLX Phi-4 reasoning fine-tuning notebook writes generated JSONL files.

The notebook currently writes to ./data/train.jsonl and ./data/valid.jsonl, but the data directory is not present in a fresh checkout and is not created before those writes.

Changes

  • Add Path("data").mkdir(exist_ok=True) before exporting JSONL files
  • Keep the existing train.jsonl and valid.jsonl output filenames
  • Limit the change to the MLX Phi-4 reasoning notebook

Validation

  • Confirmed md/02.Application/03.AdvancedReasoning/Phi4/AdvancedResoningPhi4mini/data is absent in a fresh checkout
  • Confirmed the notebook writes to ./data/train.jsonl and ./data/valid.jsonl
  • Confirmed the updated notebook creates ./data before those writes

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.

1 participant