Skip to content

fix org reference for central node#42

Merged
gadomski merged 1 commit intomainfrom
fix-centrol-repository-name
Feb 6, 2026
Merged

fix org reference for central node#42
gadomski merged 1 commit intomainfrom
fix-centrol-repository-name

Conversation

@aboydnw
Copy link
Member

@aboydnw aboydnw commented Feb 6, 2026

Root cause

config.toml had central_repository = "DevSeed Team" (no org prefix), but the CSV data on disk has developmentseed/DevSeed Team (with org prefix). When the Jinja template passed "DevSeed Team" as REPO_CENTRAL to the visualization, prepareData created nodes from the CSV with id: "developmentseed/DevSeed Team", so the lookup d.id === "DevSeed Team" failed.

This likely happened during the refactoring, probably because I removed the center node visualization. I made a followup ticket to fully remove #40
The config value was shortened to just the display name, but the CSV (which was generated earlier with the full owner/repo format) wasn't regenerated.

Two changes made:

  1. config.toml — restored the full owner/repo format: central_repository = "developmentseed/DevSeed Team". This matches the CSV data and the convention used by every other repository in the config.
  2. python/templates/index.html.jinja line 54 — changed the title display from {{ central_repository }} to {{ central_repository.split('/')[-1] }} so the heading still renders as "DevSeed Team" rather than the full path.

@aboydnw aboydnw marked this pull request as ready for review February 6, 2026 17:45
@aboydnw aboydnw requested a review from gadomski as a code owner February 6, 2026 17:45
@aboydnw aboydnw linked an issue Feb 6, 2026 that may be closed by this pull request
@gadomski gadomski merged commit 1758b20 into main Feb 6, 2026
1 check passed
@gadomski gadomski deleted the fix-centrol-repository-name branch February 6, 2026 19:44
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.

Broken visualization because of center node

2 participants