Skip to content

Portfolio: mention the Embedding Atlas constructs world - #18

Merged
jcdavis131 merged 2 commits into
masterfrom
claude/eye-jcamd-geospatial-layers-c2xpyx
Sep 23, 2026
Merged

jcdavis131 merged 2 commits into
masterfrom
claude/eye-jcamd-geospatial-layers-c2xpyx

Conversation

@jcdavis131

Copy link
Copy Markdown
Owner

This updates the copy on the Embedding Atlas flagship card to mention the new constructs layer (jcdavis131/gods-eye-view#6): every jurisdiction, district, watershed and ecoregion over a point, joined back to the physical layers by location. It is a one-line copy change in index.html.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WH1hPpf3u1xWecEhS6AHD2


Generated by Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WH1hPpf3u1xWecEhS6AHD2
@vercel

vercel Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
jcamd Ready Ready Preview Sep 22, 2026 6:12pm UTC

Request Review

Copy link
Copy Markdown
Owner Author

smoke is failing, but not because of this PR. Running python3 scripts/smoke.py on master (9a308a8) fails with the same six errors. index.html no longer links the dumbmodel.com sites that REQUIRED_LINKS expects: https://dumbmodel.com/, then hoops., pitch., gridiron., equities. and unified.dumbmodel.com/. The only one still on the page is https://hoops.dumbmodel.com, and it has no trailing slash. This PR changes one paragraph of the Atlas card and touches none of those links.

I haven't found an existing fix for this, and I haven't re-run the job: it fails the same way every time, and I reproduced it on master locally. There are two ways to fix it, and which one is right depends on what the portfolio should show:

  1. Put the game-suite links back into index.html, e.g. in the "Vector game suite" card.
  2. Or, if they were removed on purpose, trim the list:
 REQUIRED_LINKS = [
-    "https://dumbmodel.com/",
-    "https://hoops.dumbmodel.com/",
-    "https://pitch.dumbmodel.com/",
-    "https://gridiron.dumbmodel.com/",
-    "https://equities.dumbmodel.com/",
-    "https://unified.dumbmodel.com/",
+    "https://hoops.dumbmodel.com",
     "/graphify/",
     "/family/",
 ]

Tell me which one you want and I'll push it to this branch.


Generated by Claude Code

The game-suite links were removed from index.html on purpose; the smoke
list still required all six, so smoke failed on master.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WH1hPpf3u1xWecEhS6AHD2

Copy link
Copy Markdown
Owner Author

Resolved in 7389638: the game-suite links were removed on purpose, so REQUIRED_LINKS now requires only https://hoops.dumbmodel.com (option 2 above). smoke passes on this branch. Merging this PR also fixes smoke on master.


Generated by Claude Code

@jcdavis131
jcdavis131 marked this pull request as ready for review September 23, 2026 13:48
@jcdavis131
jcdavis131 merged commit 848ea8f into master Sep 23, 2026
3 checks passed

@cursor cursor Bot 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.

Approved. Cursor Bugbot was not present after the first check poll, so that signal was skipped, and no applicable approval policy required human review. No reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

This branch was successfully deployed

1 active deployment
Preview — 73896385 Deployed Sep 22, 2026 by vercel[bot]
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