insightsengineering / insightsengineering/teal.code

code shouldn't be stored as character

Aperta
#181 5 commenti 0 reazioni 0 assegnatari Vedi su GitHub
core
Lingua principale
R
Stelle
12
Fork
11
Merge medio
1g 17h
PR unite (30g)
1

Descrizione

In R, there are more appropriate structures to store the _code_ but we keep storing this as a character.

This has some indirect negative consequences of unnecessary type conversions. Right now, if I call `quenv() |> within({})` then my `` is [converted to character](https://github.com/insightsengineering/teal.code/blob/3c1a37052c775a0d4466a8dbe852db2b909f6b31/R/qenv-eval_code.R#L94-L96) and then [back again to the code](https://github.com/insightsengineering/teal.code/blob/3c1a37052c775a0d4466a8dbe852db2b909f6b31/R/qenv-eval_code.R#L36). Such behaviour is currently being adapted in `teal.data` (`eval_code.teal_data_module`) so if changed - please change there as well.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia leggendo R/qenv-eval_code.R alle righe indicate 36 e 94-96 per tracciare la conversione corrente dei caratteri intorno a quenv() |> within(). Esamina quindi l’implementazione di eval_code.teal_data_module in teal.data per confrontare il comportamento correlato. Il lavoro è completo quando la conversione non necessaria è stata corretta in modo coerente in entrambe le posizioni senza modificare il comportamento della valutazione del codice.

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

Valutazione

Stack tecnologico
r
Ambito
tooling
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.