code shouldn't be stored as character
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
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.
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({<code>}) then my <code> is converted to character and then back again to the code. Such behaviour is currently being adapted in teal.data (eval_code.teal_data_module) so if changed - please change there as well.
- Lingua principale
- R
- Stelle
- 12
- Fork
- 11
- Merge medio
- 8g 15h
- PR unite (30g)
- 2
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un 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 insightsengineering/teal.code
-
Move to PharmaverseForse già presa @llrs-roche l’ha presa 38 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
insightsengineering/teal.code#292 ·
-
[Feature Request]: High cyclomatic complexity leds to bad readabilityForse già presa @llrs-roche l’ha presa 39 giorni fa. Apertaenhancement
Difficoltà 3/5 3-5 giorni Idoneità per principianti 45/100
insightsengineering/teal.code#295 ·
-
core discussion
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
insightsengineering/teal.code#291 · 2 commenti · 1 reazione ·
-
bug core
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
insightsengineering/teal.code#275 ·
-
bug core
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
insightsengineering/teal.code#272 ·
Tutte le issue di insightsengineering/teal.code
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
mitchelloharawild/ggtime#39 ·
-
enhancement good first issue help wanted
Difficoltà 2/5 Mezza giornata Idoneità per principianti 82/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
question
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
Breaking 🏴☠️ Consistency :green_apple: :apple:
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 70/100
easystats/bayestestR#789 ·
I maintainer di solito rispondono entro 1 giorno