code shouldn't be stored as character
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
Piste de recherche
Commencez par lire R/qenv-eval_code.R aux lignes indiquées 36 et 94-96 afin de suivre la conversion actuelle des caractères autour de quenv() |> within(). Examinez ensuite l’implémentation de eval_code.teal_data_module dans teal.data pour comparer le comportement associé. Le travail est terminé lorsque la conversion inutile est corrigée de manière cohérente dans les deux emplacements, sans modifier le comportement de l’évaluation du code.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- R
- Étoiles
- 12
- Forks
- 11
- Merge moyen
- 8 j 15 h
- PR mergées (30 j)
- 2
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de insightsengineering/teal.code
-
Move to PharmaversePeut-être pris @llrs-roche l’a pris il y a 37 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
insightsengineering/teal.code#292 ·
-
[Feature Request]: High cyclomatic complexity leds to bad readabilityPeut-être pris @llrs-roche l’a pris il y a 38 jours. Ouverteenhancement
Difficulté 3/5 3-5 jours Accessibilité débutants 45/100
insightsengineering/teal.code#295 ·
-
core discussion
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
insightsengineering/teal.code#291 · 2 commentaires · 1 réaction ·
-
bug core
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
insightsengineering/teal.code#275 ·
-
bug core
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
insightsengineering/teal.code#272 ·
Toutes les issues de insightsengineering/teal.code
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
stan-dev/priorsense#150 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
xuzhougeng/wisp-science#1571 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
bennettoxford/opencodecounts#170 ·