You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Add agent discovery symlinks, HelpOnErrorParser, PEP 723 deps, DefaultAzureCredential
* Add agent discovery symlinks for fine-tuning skill
Add symlinks so coding agents (Copilot, Claude, Codex) can auto-discover
the fine-tuning skill from their conventional paths:
- .github/skills/azure-ai-fine-tuning -> ../../Skills
- .claude/skills/azure-ai-fine-tuning -> ../../Skills
- .agents/skills/azure-ai-fine-tuning -> ../../Skills
This reuses the existing comprehensive skill (SFT, DPO, RFT with graders
and tools, 8 scripts, 11 reference docs, 5 workflows) rather than creating
a separate thinner skill.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Add HelpOnErrorParser, PEP 723 deps, DefaultAzureCredential, and AI Agent Skills docs
Adopts three patterns from PR #27:
1. HelpOnErrorParser in common.py — prints full --help on invalid args
2. PEP 723 inline script metadata (# /// script) on all scripts —
enables 'uv run scripts/submit_training.py' with auto-installed deps
3. DefaultAzureCredential fallback in get_clients() — works without
API key when az CLI or Managed Identity is available
Also adds AI Agent Skills section to README with usage instructions
for Copilot (VS Code + CLI), Claude Code, and Codex agents.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Fix: scripts now use HelpOnErrorParser, fix credential fallback bug
- All 7 scripts now import and use HelpOnErrorParser from common.py
(previously added the class but scripts still used argparse.ArgumentParser)
- Fix bug in DefaultAzureCredential fallback: when base_url is set but
api_key is None and credential fails, no longer creates client with
api_key=None (which would fail silently on first API call)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Fix: DPO system message bug, cross-dir import path, dedup imports
- convert_dataset.py: Fix DPO generation to include system messages when
generating non-preferred responses from base model (was generating without
system prompt but recording it in the DPO input — distributional mismatch)
- All scripts: Add sys.path.insert for common.py import to work when
scripts are run from outside the scripts/ directory
- Clean up duplicate os/sys imports
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---------
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
0 commit comments