Skip to content

Conversation

@brtietz
Copy link
Collaborator

@brtietz brtietz commented Jun 30, 2025

Create Jupyter notebook with proof of concept code for running SAM with a green button data XML

Since the example file was from Janine's house, I haven't included it in the repository, but am open to do so after going through it in more detail to remove any PII. (I can share the file internally for the code review).

Also, open to feedback that this is more appropriate for https://github.com/NREL/SAM-analyses than a PySAM example

Copy link
Collaborator

@dguittet dguittet left a comment

Choose a reason for hiding this comment

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

This worked! Minor changes requested

"import PySAM.Utilityrate5 as utility_rate\n",
"import PySAM.Cashloan as cashloan\n",
"import PySAM.UtilityRateTools\n",
"import requests"
Copy link
Collaborator

Choose a reason for hiding this comment

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

add print(PySAM.__version__) so we can show it's been tested with the latest version

"year = 2023\n",
"\n",
"sam_api_key = \"YOUR_API_KEY\"\n",
"sam_email = \"YOUR_EMAIL\"\n",
Copy link
Collaborator

Choose a reason for hiding this comment

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

move these key variables to the very top so it's easy to set from the beginning. also, name them nrel_developer_api_key and nrel_developer_email so it's clear it's not SAM-based

@brtietz brtietz requested a review from dguittet October 24, 2025 16:30
@dguittet dguittet merged commit 1f249ca into main Oct 24, 2025
0 of 20 checks passed
@dguittet dguittet deleted the green_button_data_example branch October 24, 2025 18:02
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.

3 participants