Skip to content

Branding: Update NLR branding - #431

Merged
akeeste merged 1 commit into
MHKiT-Software:developfrom
simmsa:update_nlr_branding
Dec 17, 2025
Merged

akeeste merged 1 commit into
MHKiT-Software:developfrom
simmsa:update_nlr_branding

Conversation

@simmsa

@simmsa simmsa commented Dec 10, 2025

Copy link
Copy Markdown
Contributor

Update documentation to use National Laboratory of the Rockies per: https://www.energy.gov/eere/articles/energy-department-renames-nrel-national-lab-rockies

At this point in time https://nrel.gov is still active and https://nlr.gov does not work, so we are keeping the nrel.gov links the same.

Also the HSDS api endpoints:

/nrel/US_wave/virtual_buoy/{region}/{region}_virtual_buoy_{year}.h5
/nrel/wtk/{region.lower()}/{region}_*.h5
/nlr/wtk/{region.lower()}-5min/{region}_*.h5

In

mhkit/wave/io/hindcast/hindcast.py
mhkit/wave/io/hindcast/wind_toolkit.py

Are not updated yet, but may change in the future.

@akeeste
akeeste self-requested a review December 17, 2025 20:06

@akeeste akeeste left a comment

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.

Thanks Andrew, approved and merging

@akeeste
akeeste merged commit 323e741 into MHKiT-Software:develop Dec 17, 2025
57 checks passed
simmsa added a commit that referenced this pull request Jul 10, 2026
Update documentation to use National Laboratory of the Rockies per:
https://www.energy.gov/eere/articles/energy-department-renames-nrel-national-lab-rockies

At this point in time <https://nrel.gov> is still active and
<https://nlr.gov> does not work, so we are keeping the nrel.gov links
the same.

Also the HSDS api endpoints:

`/nrel/US_wave/virtual_buoy/{region}/{region}_virtual_buoy_{year}.h5`
`/nrel/wtk/{region.lower()}/{region}_*.h5`
`/nlr/wtk/{region.lower()}-5min/{region}_*.h5`

In

```
mhkit/wave/io/hindcast/hindcast.py
mhkit/wave/io/hindcast/wind_toolkit.py
```

Are not updated yet, but may change in the future.
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