coordsFromBbox enlarge width and height twice
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- web-dev
Direzione di ricerca
Inizia in index.js alle righe 42-43 e 50-51, dove l’issue identifica il calcolo iniziale delle dimensioni e il secondo ingrandimento. Riproduci i due esempi di console.log, quindi verifica che coordsFromBbox non raddoppi più la larghezza e l’altezza e corrisponda a coordsFromCenter per gli stessi limiti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The width calculated by coordsFromBbox() is base_width * scale *scale, which should be base_width * scale. Let check out:
console.log(abaculus.coordsFromBbox(0, 2, [-180, -85.0511, 180, 85.0511], 19000, 256))
console.log(abaculus.coordsFromCenter(0, 2, {x: 0, y: 0, w: 256, h: 256}, 19000, 256))
What I got is :
{ w: 1024, h: 1024, x: 256, y: 256 }
{ w: 512, h: 512, x: 256, y: 256 }
We have already get the right size from here: https://github.com/mapbox/abaculus/blob/master/index.js#L42-L43
But we enlarged the size again at here: https://github.com/mapbox/abaculus/blob/master/index.js#L50-L51
And x: 256, y: 256 is definitely not the center of w: 1024, h: 1024, which proves the results calculated by coordsFromBbox is wrong.
If this problem is confirmed. I will send a PR.
- Lingua principale
- JavaScript
- Stelle
- 128
- Fork
- 33
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di mapbox/abaculus
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Place markersAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 30/100
-
What does zoomTo mean？Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
-
vector.pbf blendingAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 55/100
Tutte le issue di mapbox/abaculus
Issue simili
-
internal os specific
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
frostney/GocciaScript#1688 ·
I maintainer di solito rispondono entro 1 giorno
-
🐞 bug pending triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
CircuitVerse/CircuitVerse#8031 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
I maintainer di solito rispondono entro 8 giorni
-
docs(stream-protocol): document messageMetatadata on start/finish and the message-metadata chunkForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertafactory-active task-identify-harness-labels-done task-identify-issue-type-done
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
vercel/ai#23375 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 7 giorni