DOI-USGS / DOI-USGS/dataretrieval-python

enums for site type, county, etc.

未关闭
#98 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Python
星标
266
派生
63
平均合并
1 天 20 小时
30 天内合并 PR
19

描述

How do we feel about using [`enum`](https://docs.python.org/3.8/library/enum.html) for [site type](https://github.com/DOI-USGS/dataretrieval-python/blob/87351a13b7c7c1cf035e9a359addf289d14f5e50/dataretrieval/nwis.py#L609-L613), [county code](https://github.com/DOI-USGS/dataretrieval-python/blob/87351a13b7c7c1cf035e9a359addf289d14f5e50/dataretrieval/nwis.py#L583-L586C15), etc., instead of `str`?

It would fit nicely with:
- #97

贡献指南

打开贡献指南

调研方向

首先阅读链接网站中 dataretrieval/nwis.py 的站点类型和县代码定义,然后查看 #97 中的相关提案。确定哪些值应成为 enum,以及该变更如何与现有 API 适配;当范围和 enum 设计达成一致时即视为完成,因为此 issue 目前要求的是讨论,而不是指定实现。

由索引模型根据 Issue 内容生成。

评估

技术栈
python
领域
api
Issue 类型
重构
难度
5/5
预计耗时
一周以上
活跃度
停滞
描述清晰度
需要澄清
新手友好度
25/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。