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.

Sign up for GitHub

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

Jump to bottom

WV-3368 backup eic links #5495

Merged
merged 2 commits into from
Oct 10, 2024
Merged

WV-3368 backup eic links #5495

merged 2 commits into from
Oct 10, 2024

Conversation

PatchesMaps
Copy link
Contributor

@PatchesMaps PatchesMaps commented Oct 7, 2024 •
edited
Loading

Description

Create backup EIC links for NOAA-20 and NOAA-21

How To Test

  1. git checkout WV-3368-backup-eic-links
  2. npm run watch
  3. Test these scenarios: 1, 2, 3, 4

PR Submission Checklist

This is simply a reminder of what we are going to look for before merging your code.

  • I have read the CONTRIBUTING doc
  • I have added necessary documentation (if applicable)
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • Any dependent changes have been merged and published in downstream modules (if applicable)

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

@PatchesMaps
legacy mode eic
630b181
@PatchesMaps
Refactor layer-data-eic.js: Remove unused thermal anomaly layers
602b9eb 
This commit removes the unused thermal anomaly layers, VIIRS_NOAA21_Thermal_Anomalies_375m_All and VIIRS_NOAA20_Thermal_Anomalies_375m_All, from the layersToMeasure, layerPixelData, and bestDates arrays in the layer-data-eic.js file. These layers are no longer needed and can be safely removed.
@PatchesMaps PatchesMaps self-assigned this Oct 7, 2024
@minniewong minniewong added this to the v4.50.0 milestone Oct 9, 2024
@PatchesMaps PatchesMaps merged commit d76da00 into develop Oct 10, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@minniewong minniewong minniewong approved these changes

@christof-wittreich christof-wittreich christof-wittreich approved these changes

@Tomcariello Tomcariello Awaiting requested review from Tomcariello

Assignees

@PatchesMaps PatchesMaps

Labels
None yet
Projects
None yet
Milestone
v4.50.0
Development

Successfully merging this pull request may close these issues.
3 participants
@PatchesMaps @christof-wittreich @minniewong