Skip to content

Wind Modeling using PySAM Wind Model#59

Merged
bmeyers merged 13 commits into
mainfrom
pysam_new_edits
Jan 5, 2026
Merged

Wind Modeling using PySAM Wind Model#59
bmeyers merged 13 commits into
mainfrom
pysam_new_edits

Conversation

@qnguyen345
Copy link
Copy Markdown
Collaborator

Contains the results and script to model wind from nsrdb weather data and uswtdb metadata.

run_pysam.py: Models wind data from nsrdb and uswtdb metadata on a single turbine level.
agg-pysam-wind-powerplant-predictions-node.py: Aggregates the turbine level data by node 1) for all nodes and 2) for only CA nodes.

@qnguyen345 qnguyen345 requested a review from kperrynrel April 2, 2025 17:55
Copy link
Copy Markdown
Collaborator

@kperrynrel kperrynrel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we set the pysam_geopanel.csv and ca_pysam_geopanel.csv at 2 sig figs? Also let's change the aggregation script to something more generic (I'll delete my copy in my branch and we can just use yours). Maybe like ' agg-powerplant-predictions-node.py'?

Copy link
Copy Markdown
Collaborator

@kperrynrel kperrynrel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to fold the uswtdb_metadata file into uswtdb_pysam_sim file? Looks like there may be some field replication here, and we could just make one main file for all the metadata

Comment thread data/run_pysam.py Outdated
print("Modeled:", filename)
pass
elif filename =="9q713t_35.108196_-118.359886":
pass
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's going on with this edge case? Just wondering if we can eliminate it or not

Copy link
Copy Markdown
Collaborator Author

@qnguyen345 qnguyen345 Apr 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I was debugging something in the code and I forgot to take it out. I removed this if statement and reran it for that site and it ran fine

@bmeyers bmeyers requested a review from dchassin April 9, 2025 17:19
Copy link
Copy Markdown
Collaborator

@dchassin dchassin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add copied/scaled/rounded the data into the geodata folder

@dchassin dchassin mentioned this pull request May 29, 2025
@bmeyers bmeyers merged commit b836459 into main Jan 5, 2026
1 check failed
@bmeyers bmeyers deleted the pysam_new_edits branch January 5, 2026 21:14
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.

4 participants