Skip to content

Migrated models from global to per directory - #197

Open
gargrahul wants to merge 1 commit into
ROCm:developfrom
gargrahul:gargrahul/discover_model_migration
Open

Migrated models from global to per directory#197
gargrahul wants to merge 1 commit into
ROCm:developfrom
gargrahul:gargrahul/discover_model_migration

Conversation

@gargrahul

Copy link
Copy Markdown
Collaborator

This migration moves model definitions from the root models.json file to per-directory models.json files within each script directory.

Changes:

  • Migrated 132 models to 20 per-directory models.json files
  • Kept 1 model in root models.json (primus_train has get_models_json.py)
  • Added tools/migrate_to_dir_models.py migration script

Path translations applied:

  • dockerfile: docker/X -> ../../docker/X
  • scripts: scripts//file.sh -> file.sh
  • dockercontext: kept as-is (project-root-relative)

This migration moves model definitions from the root models.json file
to per-directory models.json files within each script directory.

Changes:
- Migrated 132 models to 20 per-directory models.json files
- Kept 1 model in root models.json (primus_train has get_models_json.py)
- Added tools/migrate_to_dir_models.py migration script

Path translations applied:
- dockerfile: docker/X -> ../../docker/X
- scripts: scripts/<dir>/file.sh -> file.sh
- dockercontext: kept as-is (project-root-relative)
Copilot AI lite review requested due to automatic review settings August 6, 2026 19:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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