Skip to content

restored image paths and colab pointer#4

Merged
amithjkamath merged 1 commit into
ubern-mia:mainfrom
amithjkamath:main
May 13, 2026
Merged

restored image paths and colab pointer#4
amithjkamath merged 1 commit into
ubern-mia:mainfrom
amithjkamath:main

Conversation

@amithjkamath

Copy link
Copy Markdown
Collaborator

Fixing path issues for README file;

Copilot AI review requested due to automatic review settings May 13, 2026 17:01
@amithjkamath amithjkamath merged commit ec086c3 into ubern-mia:main May 13, 2026
1 of 2 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Restores image references in the training-models README to point at the in-repo experiment folders (rather than the gitignored results/vN/ directories), updates the documentation to reflect the split between tracked plot assets and ignored artifacts, unignores those experiment folders so the referenced images render, and fixes the Jupyter Book branch from master to main.

Changes:

  • Rewrite README "Results directory layout" section and image links to use dermamnist_vN_*/ paths.
  • Remove .gitignore entries that were excluding the per-version experiment folders; add *.onnx.data.
  • Fix _config.yml branch reference from master to main so Colab/launch links resolve.

Reviewed changes

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

File Description
training-models/README.md Updates layout description and image paths to in-repo experiment folders.
training-models/_config.yml Switches Jupyter Book repo branch from master to main.
.gitignore Stops ignoring per-version experiment dirs; ignores *.onnx.data.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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