Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Affine trans. matrix element ordering for `ST_Affine`

Aperta Adatta ai principianti
#289 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
85/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
r
Ambito
databases

Direzione di ricerca

Individua la funzione .trans_f() nel codice sorgente del pacchetto e sostituisci lo splice !!!v con c(t(m[, -ncol(m)]), m[, ncol(m)]) per corrispondere all'ordine ST_Affine di DuckDB. Esegui i test di trasformazione spaziale (inclusi i due nuovi) per confermare che una rotazione non è più trasposta.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

  • .trans_f() hands the matrix to ST_Affine() with !!!v, which splices it column by column. For rbind(c(a, b, xoff), c(d, e, yoff)) the call becomes ST_Affine(geometry, a, d, b, e, xoff, yoff), but DuckDB's 2D signature is ST_Affine(geom, a, b, d, e, xoff, yoff). So the 2 × 2 part is transposed and a rotation comes out turned the other way. The two new tests use a translation and a diagonal scale, which give the same result either way. c(t(m[, -ncol(m)]), m[, ncol(m)]) would give DuckDB the order it expects, in 2D and 3D.

Cuiwei

Originally posted by @CuiweiG in #251

Following the above comment, should be an easy fix.

Lingua principale
R
Stelle
37
Fork
10
Merge medio
22h 46m
PR unite (30g)
20

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di HelenaLC/spatialdataR

Tutte le issue di HelenaLC/spatialdataR

Issue simili

Altre issue su R

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.