cmu-delphi/delphi-epidata

Refactor `csv_importer.py` and `csv_to_database.py` and `covidcast_row.py`

Aperta

#1078 aperta il 3 feb 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (67 fork)auto 404
acquisitioncode healthgood first issuehelp wantedrefactor

Metriche repository

Star
 (104 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

A number of refactors can be done here:

  • csv_importer and csv_to_database are tightly coupled, but with low cohesion; would probably be best if merged together
  • csv_importer validation could likely be handled by Pandas exceptions
  • csv_importer can exit early when detecting a Pandas exception and save time
  • csv_importer can probably share validation with covidcast_row

Guida contributor

Direzione di ricerca
Analizza l'accoppiamento stretto tra csv importer e csv to database e proponi di unirli. Valuta come le eccezioni di Pandas possono sostituire la validazione personalizzata e identifica i pattern di validazione condivisi con covidcast row.
Tech stack
python
Dominio
backendapidata
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
PythonPandas
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60