Skip to content

Add Missing Regional Capital Cost Multipliers#233

Merged
kennedy-mindermann merged 1 commit intomainfrom
km/missing-rm
Jun 10, 2025
Merged

Add Missing Regional Capital Cost Multipliers#233
kennedy-mindermann merged 1 commit intomainfrom
km/missing-rm

Conversation

@kennedy-mindermann
Copy link
Copy Markdown
Collaborator

Summary

This PR adds regional multipliers for some (32) counties that were missing. I just took the reg_cap_cost_mult from the nearest county (looking at a map and manually deciding which county was closest).

Here's the county mapping I used: missing_counties_map.csv

County-level runs that include CO, MO, VA, NY, or RI should no longer fail.

Technical Details

Implementation notes

Additional changes (if any)

Switches added/removed/changed (if any)

Issues resolved (if any)

Known incompatibilities (if any)

Relevant sources or documentation (if any)

Is the model cleaner/better/faster/smaller than before? If so, how?

Validation / Comparison Report

Link to report and/or example plots

(if pertinent) How big is the default run folder (runs/{base}_ref_seq/) before and after the change?

(if pertinent) What is the run time for ref_seq on the same machine before and after the change?

@kennedy-mindermann kennedy-mindermann merged commit 39a3e3f into main Jun 10, 2025
1 check passed
@kennedy-mindermann kennedy-mindermann deleted the km/missing-rm branch June 10, 2025 21:20
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