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

Online access arctic dem #799

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

trey-stafford
Copy link
Contributor

@trey-stafford trey-stafford commented Aug 30, 2023

Description

Add an online-only Arctic DEM. Related to #797

This adds a new "Arctic DEM" layer to "Terrain models/Internet required". The layer is grayscale but does have a legend. Unfortunately the legend does not have a label indicating units.

This layer is configured to use the arcgismapserver provider, which seems to provide a similar experience as WMS. The arcgis service does provide a WMS interface, it provides the same legend & grayscale data view. In both cases the identify tool and value tool cannot be used to inspect data values. Is this worth adding?

latestScreen

Checklist

If an item on this list is done or not needed, check it with [x] or click the
checkbox.

  • The PR description links to issues that it resolves with closes #{issue_number}
  • Config lockfile updated (inv config.export > qgreenland/config/cfg-lock.json)
  • Environment lockfile updated if needed (conda-lock)
  • Version bumped if needed (bumpversion (major|minor|patch|prerelease|build))
  • CHANGELOG.md updated (for user-facing changes)
  • Documentation updated if needed
  • New unit tests if needed

@trey-stafford trey-stafford mentioned this pull request Aug 30, 2023
4 tasks
@trey-stafford trey-stafford marked this pull request as ready for review August 31, 2023 15:24
Copy link

@MattF-NSIDC MattF-NSIDC left a comment

Choose a reason for hiding this comment

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

Looks great! Pleased with how easy the new online access provider was to add.

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