Skip to content

bugfix: Update tableau visualization script to correctly parse get_zonemap() output#132

Open
kodiobika wants to merge 1 commit into
mainfrom
ko/bugfix/tableau_viz
Open

bugfix: Update tableau visualization script to correctly parse get_zonemap() output#132
kodiobika wants to merge 1 commit into
mainfrom
ko/bugfix/tableau_viz

Conversation

@kodiobika

@kodiobika kodiobika commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Summary

The Tableau visualization script was previously treating the outputs of reeds.io.get_zonemap() as having a nameless index, but it now has a name (r). As a result, the copying of relevant shapefiles was silently failing resulting in the visualization outputs containing no shapefiles.

Validation, testing, and comparison report(s)

I ran the script for the WY_county scenario and it now produces transmission_endpoints and US_PCA shapefiles.

General information to guide review

  • Zero impact on results of default case
  • No large data file(s) added/modified
  • No substantive impact on runtime for full-US reference case
  • No substantive impact on folder size for full-US reference case
  • No change to process flow (runreeds.py, reeds/core/solve/solve.py)
  • No change to code organization
  • No change to package requirements (environment.yml or Project.toml)

Did you use LLM tools (chatbot or copilot) in the preparation of this PR? If so, describe how

No

Tag points of contact here if you would like additional review of the relevant parts of the model

@kodiobika kodiobika requested a review from ahamilton5 June 26, 2026 19:52
@kodiobika kodiobika self-assigned this Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant