eduaguilera/whep

spatialize: dead helpers .normalize_to_cropland and .get_area_code_from_grid

Aperta

#225 aperta il 22 lug 2026

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)R (3 fork)auto 404
area:spatializeclean codegood first issuehelp wantedmechanicalno-data-neededpriority:low

Metriche repository

Star
 (1 stella)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

What

.normalize_to_cropland (R/spatialize.R:924) and .get_area_code_from_grid (:633) are defined but never called anywhere in R/ or tests/. Also type_count <- dplyr::n_distinct(...) (:199) is computed and immediately discarded (comment implies a message was intended but none is emitted).

Suggested fix / check

Remove the dead helpers and the unused computation (or wire up the intended message).

From deep static audit (2026-07-22), parallel subsystem review pass. Deduplicated against then-open issues. Not verified by running unless stated.

Guida contributor

Direzione di ricerca
Cerca nell'intero repository, inclusi R/ e tests/, per confermare che .normalize to cropland e .get area code from grid non vengono mai chiamati. Verifica anche la variabile inutilizzata type count. Quindi rimuovi il codice morto e, se esiste un messaggio previsto, collega o rimuovi anche quello.
Tech stack
r
Dominio
data
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitR
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80