Skip to content

Fix encoded statistical model detail links - #10

Merged
codoom1 merged 1 commit into
mainfrom
agent/fix-encoded-model-detail
Jul 24, 2026
Merged

Fix encoded statistical model detail links#10
codoom1 merged 1 commit into
mainfrom
agent/fix-encoded-model-detail

Conversation

@codoom1

@codoom1 codoom1 commented Jul 24, 2026

Copy link
Copy Markdown
Owner

What changed

  • Decode model names that remain URL-encoded after Flask processes the path.
  • Apply normalization consistently to model details, interpretation guides, and interpretation downloads.
  • Reuse the same path-decoding helper for model-group routes.
  • Add regression coverage for double-encoded model URLs.

Root cause

Some deployed links arrive double-encoded. Flask decodes the path once, leaving a lookup value such as Linear%20Regression instead of the model database key Linear Regression, which caused the site to render “Model not found.”

Impact

Model detail pages and their interpretation/download actions now resolve correctly for both ordinary and previously double-encoded links.

Validation

  • python -m pytest -q tests/ — 118 passed
  • Focused main-route tests — 22 passed
  • Strict Python undefined-name/syntax lint — passed
  • git diff --check — passed

@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
statistical-model-suggester Ready Ready Preview, Comment Jul 24, 2026 10:44pm

@codoom1
codoom1 marked this pull request as ready for review July 24, 2026 22:49
Copilot AI review requested due to automatic review settings July 24, 2026 22:49
@codoom1
codoom1 merged commit 9e011a1 into main Jul 24, 2026
7 checks passed

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 was unable to review this pull request because the user who requested the review has reached their quota limit.

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