insightsengineering / insightsengineering/teal.code
code shouldn't be stored as character
- 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