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

chore: projection test app #9200

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

chore: projection test app #9200

wants to merge 1 commit into from

Conversation

Pessimistress
Copy link
Collaborator

For #9199

Change List

  • Add test app for projection precision & base map alignment

@coveralls
Copy link

Coverage Status

coverage: 91.644%. remained the same
when pulling d3cff38 on x/globe-test-app
into da0c1f9 on master.


const opts = VIEWS[viewMode];

const pointData = React.useMemo(() => makePointGrid(opts), [viewMode]);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would be nice to include a finer grid around zoom 12 to see how much we deviate from the adaptive projection in maplibre

Copy link
Collaborator

@chrisgervang chrisgervang left a comment

Choose a reason for hiding this comment

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

Let's say projection were a little off between the basemap and deck, which parameters would someone tune to align them? MapView's fov?

initializeState({device}: LayerContext) {
const canvas = device.canvasContext?.canvas as HTMLCanvasElement;
if (canvas) {
const container = appendSVGElement(canvas.parentElement!, 'svg');
Copy link
Collaborator

Choose a reason for hiding this comment

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

Interesting use of a deck layer.
Is it correct to say this renders a 2D grid with SVG over the canvas? I'm assuming this only works since pitch is 0?


const layers = [
// Reference grid when base map is not available (non-geo)
!opts.baseMap &&
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could it be useful to visually compare the deck lines with the basemap lines by overlapping them and rendering them in different colors?

I'm curious about how to perform a projection check

@felixpalmer felixpalmer mentioned this pull request Dec 4, 2024
36 tasks
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.

4 participants