Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/hurricane layers #74

Merged
merged 8 commits into from
Jun 6, 2024
Merged

Feature/hurricane layers #74

merged 8 commits into from
Jun 6, 2024

Conversation

lstillwe
Copy link
Contributor

@lstillwe lstillwe commented May 30, 2024

This feature add hurricane track layers support. I am not aware of any issues regarding this feature, except as listed in issue #71 (as requested by b.blanton).
The new code in files in utils/hurricane (track.js, config.js, and time.js) were originally stolen from the website that uses the MetGet api. The MetGet api is used to retrieve the hurricane track data in GeoJson format. It returns features for the cone of uncertainty, the hurricane track line, and the advisory point locations.
This feature uses a new state variable in map-context called 'hurricaneTrackLayers'. The updates touched a few components, including the new StormLayers component and the ControlPanel and HurricaneList tray.
When a tropical storm adcirc layer is added to the map, the associated hurricane track layers are added automatically and they are removed when the associated adcirc layers are removed.
Please note that we will not be able to actually test this with adcirc storm layers until we can display them with the layer filtering form that Phil is working on. I have tested this feature thus far by faking with synoptic type layers

@lstillwe lstillwe requested a review from mbwatson May 30, 2024 16:03
Copy link
Member

@mbwatson mbwatson left a comment

Choose a reason for hiding this comment

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

👍 looks good to me!

@lstillwe lstillwe merged commit bc41669 into main Jun 6, 2024
@lstillwe lstillwe deleted the feature/hurricane-layers branch June 6, 2024 18:10
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