smart-on-fhir/cumulus-etl

Stop stripping ¿ from notes in general

Ouverte

#301 ouverte le 14 mars 2024

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Python (8 forks)auto 404
enhancementgood first issuei2b2

Métriques du dépôt

Stars
 (25 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

We saw some artifacts in our BCH i2b2 data and started stripping ¿ characters broadly. But that's a little hokey.

https://github.com/smart-on-fhir/cumulus-etl/blob/e551495c3514ad86cfb25f3063346ec88f7d8c73/cumulus_etl/fhir/fhir_utils.py#L221

We should limit that kind of note manipulation to the i2b2 code specifically, ideally.

Guide contributeur

Direction de recherche
Localisez le fichier fhir utils.py et la fonction qui supprime ¿. Identifiez l'endroit où se trouve le code spécifique à i2b2 et déplacez y la logique de suppression, en vous assurant qu'elle ne s'applique pas à toutes les notes.
Stack technique
python
Domaine
backenddata engineering
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPython
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75