Skip to content

Fix links to model-zoo browser#782

Closed
jabley wants to merge 2 commits intoProject-MONAI:devfrom
jabley:patch-1
Closed

Fix links to model-zoo browser#782
jabley wants to merge 2 commits intoProject-MONAI:devfrom
jabley:patch-1

Conversation

@jabley
Copy link
Copy Markdown

@jabley jabley commented Apr 20, 2026

Fixes #769 .

Description

Fix links in README to hosted model zoo browser

Status

Ready

Summary by CodeRabbit

  • Documentation
    • Updated links to model browsing resources in Getting Started and Links sections to reference the latest documentation locations.

Signed-off-by: James Abley <james.abley@gmail.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 20, 2026

Walkthrough

Updated two broken URLs in README.md, replacing https://monai.io/model-zoo with the corrected https://project-monai.github.io/model-zoo.html#/ to fix a non-functioning model zoo link referenced in the "Getting Started" and "Links" sections.

Changes

Cohort / File(s) Summary
Documentation URL Fix
README.md
Replaced two broken model zoo URLs (https://monai.io/model-zoo) with corrected endpoint (https://project-monai.github.io/model-zoo.html#/) in Getting Started and Links sections.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A broken link has found its way,
To brighter paths, hip-hip-hooray!
The zoo now blooms in files anew,
Where models dance in vibrant view! 🌟

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Fix links to model-zoo browser' directly matches the main changeset which updates two README URLs pointing to the model-zoo browser resource.
Description check ✅ Passed The description provides issue reference (#769), a clear description of changes, and status update. However, it lacks completion of the description template including required checkboxes and several optional sections.
Linked Issues check ✅ Passed The PR directly addresses issue #769 by updating the README links from the broken https://monai.io/model-zoo URL to https://project-monai.github.io/model-zoo.html#/, which fixes the reported inaccessibility of the model-zoo browser page.
Out of Scope Changes check ✅ Passed All changes are in-scope: only two README URL references were updated to resolve the linked issue, with no unrelated modifications to code, configuration, or other files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
README.md (1)

8-8: Optional: use a shared Markdown reference link to prevent future URL drift.

You now maintain the same URL in two places; a single reference keeps them in sync.

Suggested refactor
-To browse the available models, please see https://project-monai.github.io/model-zoo.html#/.
+To browse the available models, please see [MONAI model zoo browser][model-zoo-browser].

@@
-- MONAI model zoo browser: https://project-monai.github.io/model-zoo.html#/
+- MONAI model zoo browser: [MONAI model zoo browser][model-zoo-browser]
+
+[model-zoo-browser]: https://project-monai.github.io/model-zoo.html#/

Also applies to: 47-47

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 8, Replace the duplicated inline URL text
"https://project-monai.github.io/model-zoo.html#/" in README.md with a single
markdown reference-style link (e.g., [model-zoo]) wherever it appears and add a
single reference definition at the bottom of the file (e.g., [model-zoo]:
https://project-monai.github.io/model-zoo.html#/) so both occurrences reuse the
same reference and avoid URL drift.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@README.md`:
- Line 8: Replace the duplicated inline URL text
"https://project-monai.github.io/model-zoo.html#/" in README.md with a single
markdown reference-style link (e.g., [model-zoo]) wherever it appears and add a
single reference definition at the bottom of the file (e.g., [model-zoo]:
https://project-monai.github.io/model-zoo.html#/) so both occurrences reuse the
same reference and avoid URL drift.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c3143919-9255-4d32-831c-cfd93fad7c86

📥 Commits

Reviewing files that changed from the base of the PR and between d86a119 and 3b34bd1.

📒 Files selected for processing (1)
  • README.md

Signed-off-by: Eric Kerfoot <17726042+ericspod@users.noreply.github.com>
@ericspod
Copy link
Copy Markdown
Member

Hi @jabley sorry about this PR, it looks like the fixes for the URLs were included in a previous PR that I just merged now. I am seeing no files changed now after resolving conflicts so I'm afraid there's nothing to merge. Thanks for contributing though!

@ericspod ericspod closed this Apr 27, 2026
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.

Zoo page cannot be reached

2 participants