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

Raster satellite blurred in Terrain3D at vertical field of view < 15 deg #4940

Closed
wipfli opened this issue Oct 31, 2024 · 3 comments · Fixed by #4779
Closed

Raster satellite blurred in Terrain3D at vertical field of view < 15 deg #4940

wipfli opened this issue Oct 31, 2024 · 3 comments · Fixed by #4779
Labels
bug Something isn't working PR is more than welcomed Extra attention is needed

Comments

@wipfli
Copy link
Contributor

wipfli commented Oct 31, 2024

maplibre-gl-js version:

d3f2bca

browser:

Chrome on ubuntu

Steps to Trigger Behavior

  1. go to https://jsbin.com/muvadukome/edit?html,output (uses latest main after Add setVerticalFieldOfView() to public API #4914)
  2. move fov slider to 5 deg
  3. see how raster satellite get blurred

Link to Demonstration

https://jsbin.com/muvadukome/edit?html,output

image

Expected Behavior

Raster satellite stays sharp also at low fov

Actual Behavior

Raster satellite tiles are blurred
@HarelM
Copy link
Collaborator

HarelM commented Oct 31, 2024

CC: @NathanMOlson can you please check this?

@HarelM HarelM added bug Something isn't working PR is more than welcomed Extra attention is needed labels Oct 31, 2024
@birkskyum
Copy link
Member

birkskyum commented Oct 31, 2024 •
edited
Loading

Appear to be resolved, at least partially, by:

@NathanMOlson
Copy link
Contributor

This looks like a tile selection problem. It looks a lot better in the lodfix_globe branch.

Left: main, right: lodfix_globe

image image

@birkskyum birkskyum mentioned this issue Oct 31, 2024
Merged
@birkskyum birkskyum mentioned this issue Nov 4, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working PR is more than welcomed Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix level of detail at high pitch NathanMOlson/maplibre-gl-js
4 participants
@HarelM @NathanMOlson @wipfli @birkskyum