Skip to content

NREL to NLR in UI forms - #2270

Merged
cpaulgilman merged 3 commits into
patchfrom
nrel-ui-fixes
Sep 30, 2026
Merged

cpaulgilman merged 3 commits into
patchfrom
nrel-ui-fixes

Conversation

@cpaulgilman

Copy link
Copy Markdown
Collaborator

Description

Removes some remaining references to NREL in UI forms.

Also:

  • Remove API key and "sam.support@nrel.gov" from WIND Toolkit download sample script.
  • Remove API key and email from WIND Toolkit Download macro (macro context has access to internal SAM key).

Fixes #2269

Checklist

  • requires help revision and I added that label
  • adds, removes, modifies, or deletes variables in existing compute modules
  • adds a new compute module
  • changes defaults
  • I've tagged this PR to a milestone

Remove reference to sam.support@nrel.gov.

Remove macro because it requires an API key that we don't expect SAM users to need. Use LK sample file instead of macro.
Remove email and API key parameters because "<SAMAPIKEY>" and "<USEREMAIL>" are replaced by SAM's key and user email in this context.
@cpaulgilman cpaulgilman added this to the SAM 2026 Release patch 1 milestone Sep 29, 2026
@cpaulgilman cpaulgilman self-assigned this Sep 29, 2026
@cpaulgilman
cpaulgilman merged commit 3cd4af6 into patch Sep 30, 2026
6 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.

"NREL" on some input pages

2 participants