cmu-delphi/delphi-epidata

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

开放

#1,078 创建于 2023年2月3日

在 GitHub 查看
 (0 条评论) (0 个反应) (0 位负责人)Python (67 个派生)auto 404
acquisitioncode healthgood first issuehelp wantedrefactor

仓库指标

星标
 (104 个星标)
PR 合并指标
 (平均合并 63天 20小时) (30 天内合并 11 个 PR)

描述

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

贡献者指南

研究方向
分析 csv importer 和 csv to database 之间的紧密耦合，并提出合并方案。研究如何使用 Pandas 异常替代自定义验证，并识别与 covidcast row 共享的验证模式。
技术栈
python
领域
backendapidata
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
半天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
新近可参与
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
PythonPandas
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60