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

WV-2939 EIC carbon #4773

Merged
merged 1 commit into from
Oct 31, 2023
Merged

WV-2939 EIC carbon #4773

merged 1 commit into from
Oct 31, 2023

Conversation

Tomcariello
Copy link
Contributor

Description

At EIC, the Carbon Monoxide imagery on Worldview is reverting to October 6th even though there is more recent imagery available. This is because there has been no imagery for about 2 weeks & the EIC tile check function only checks the most recent 7 days before applying a known good fallback date. This problem is fixed by updating the fallback dates for each layer used in the EIC. You can confirm that this layer reverts to October 6 using this link.

Fixes # wv-2939

How To Test

  • git checkout wv-2939-eic-carbon
  • npm run watch
  • Load the same link as above on your local instance.
  • Confirm that the dates reverts to October 17th instead of October 6th.
  • Review the notes in the ticket & check that the other EIC panels are working correctly.

Merging

Please use the squash and merge commit method unless each commit in your branch is vital to the commit history of main.

@nasa-gibs/worldview

@Tomcariello Tomcariello marked this pull request as ready for review October 31, 2023 18:43
@minniewong minniewong modified the milestone: v4.18.0 Oct 31, 2023
Copy link
Contributor

@minniewong minniewong left a comment

Choose a reason for hiding this comment

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

yay! thanks!

@minniewong minniewong added this to the v4.18.0 milestone Oct 31, 2023
@Tomcariello Tomcariello merged commit 2deee2e into develop Oct 31, 2023
4 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.

3 participants