Skip to content

Organize input params#9

Merged
alxsmith merged 15 commits intomainfrom
6-organize-input-params
Aug 26, 2025
Merged

Organize input params#9
alxsmith merged 15 commits intomainfrom
6-organize-input-params

Conversation

@alxsmith
Copy link
Copy Markdown
Contributor

@alxsmith alxsmith commented Aug 25, 2025

Organized input parameters into groups, define functions to read from yaml

NOTE - i merged in other feature branches so it looks like a lot of changes but you should only focus on data/sample.yaml and params.py for this PR

We can have pre-packaged yamls with defined starting inputs. those are called by passing a run_name which is the file name for the yaml. it will auto populate the input classes and then join them into a single InputParams class nested by group

Still not sure exactly how we want to handle our timeseries values. I updated the type tags to DFs from Seiries but i think the NPA one will make more sense once we start defining that class

@alxsmith alxsmith linked an issue Aug 25, 2025 that may be closed by this pull request
@alxsmith alxsmith requested a review from karpdave August 25, 2025 18:16
pipeline_replacement_lifetime: float
pipeline_maintenance_cost_pct: float
baseline_non_lpp_gas_ratebase_growth: float
baseline_gas_lpp_costs_per_year: float
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.

Dupe with TimeSeriesParams: gas_bau_lpp_costs_per_year

@karpdave
Copy link
Copy Markdown
Contributor

karpdave commented Aug 25, 2025 via email

@alxsmith alxsmith merged commit 8b6245b into main Aug 26, 2025
0 of 7 checks passed
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.

Organize input params

2 participants