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

GrIMP ice velocity layers #749

Merged
merged 25 commits into from
Aug 23, 2023
Merged

GrIMP ice velocity layers #749

merged 25 commits into from
Aug 23, 2023

Conversation

trey-stafford
Copy link
Contributor

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

Description

Closes #663.

Adds annual GrIMP ice velocity mosaic layers for 2021 under the "Glaciology/Ice sheet velocity/GrIMP" group:

  • Annual ice sheet velocity vectors 2021
  • Annual ice sheet velocity magnitude 2021 (200m)
  • Annual ice sheet velocity x component 2021 (200m)
  • Annual ice sheet velocity y component 2021 (200m)

In total, these GrIMP layers add ~342M to the uncompressed core package. Compressed, the QGreenland core package is ~3.1G.

Styling the ice velocity vectors

I applied the style from the wind vectors layer to get started. This worked fine, but led me to realize that our wind vectors have a colormap that's inverted compared to the raster representing wind speed. This is a conflict when the two layers are visualized together. That said, the fact that the colormap is inverted makes it easier to see the vectors on top of the velocity data!

I went ahead and reversed the color map for the ice velocity vectors and forced it to match the colormap for the ice velocity magnitude raster layer. This works pretty well, but visualizing the two together is problematic because the vectors blend in with the background raster and aren't visible. Maybe an outline around the vectors would be nice, but I haven't figured out how to do that yet.

Further, I downsampled the input data for creating the wind vectors to 1.5km. This makes rendering much faster than using the full resolution data and I think it looks better too. But the downside here is that areas with major changes in velocity result in intermediate-valued vectors that, when overlaid with the raster, look like smaller/larger values than the underlying raster. Maybe a better approach would be to create vectors at full-resolution, and then 'thin' the resulting vector dataset to reduce the number of lines actually displayed?

Based on how we decide to proceed here, we should apply similar styling to the wind vectors layer.

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 marked this pull request as ready for review August 22, 2023 15:39
@trey-stafford
Copy link
Contributor Author

trey-stafford commented Aug 22, 2023

I just took a look at Quantarctica, and its MEaSUREs flow vectors do not use color to represent magnitude. Just the size (thickness and length) of the vectors themselves. Maybe that's a better solution?

latestScreen

The color mapping for magnitude makes it difficult to represent the vectors on
top of other layers with conflicting color maps.
Not required for styling the data any longer. Saves us some space too (GrIMP
package size reduced to 333M from 342).
trey-stafford added a commit that referenced this pull request Aug 22, 2023
Magnitude is represented by the size of the vector lines. The color mapping was
causing conflicts with other layers (like the raster of annual wind speed). This
problem was discovered in: #749
@trey-stafford trey-stafford mentioned this pull request Aug 22, 2023
7 tasks
@MattF-NSIDC
Copy link
Member

Just the thickness of the vectors themselves

It looks like they're using both thickness and length.

Copy link
Member

@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.

LGTM. Couple suggestions!

@trey-stafford trey-stafford merged commit b1f25a5 into main Aug 23, 2023
3 checks passed
@trey-stafford trey-stafford deleted the grimp-velocity-vectors branch August 23, 2023 19:11
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.

update ice velocity data
2 participants