False positive invalid condition ID.
@dilpath 已经在做这个了。
开始于 2020年11月10日。
评估
这个 Issue 还没有评估数据。
描述
Which problem would you like to address? Please describe.
Linting complains about invalid condition IDs. I believe the error is due to the fact that the condition id is numeric only. Ideally the error would be a bit more informative what the actual issue is with the condition ID.
Describe the solution you would like
Change the spec to be more accurate about identifier restrictions or change linting to be more permissive about identifiers.
Describe alternatives you have considered
Don't use numbers in identifiers.
Additional context
Can be checked via check_condition_df(pd.DataFrame({petab.CONDITION_ID: ['0']}).set_index(petab.CONDITION_ID), None).
Not sure how much sense it makes to print line numbers when operating with a DataFrame.
Typehints in check_condition_df are inconsistent with the function signature, i.e., sbml_model is not and optional argument.
- 主要语言
- Python
- 星标
- 18
- 派生
- 9
- PR 合并指标
- 30 天内没有已合并 PR
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
PEtab-dev/libpetab-python 的其他 Issue
-
难度 5/5 一周以上 新手友好度 25/100
PEtab-dev/libpetab-python#518 ·
-
validation
难度 5/5 一周以上 新手友好度 35/100
PEtab-dev/libpetab-python#494 · 1 条评论 ·
-
难度 4/5 3-5 天 新手友好度 35/100
PEtab-dev/libpetab-python#466 ·
-
enhancement
难度 3/5 1-2 天 新手友好度 45/100
PEtab-dev/libpetab-python#453 ·
-
documentation
难度 3/5 1-2 天 新手友好度 45/100
PEtab-dev/libpetab-python#439 ·
查看 PEtab-dev/libpetab-python 的全部 Issue
相似的 Issue
-
link-check link-check:sphinx-theme
难度 2/5 1-3 小时 新手友好度 72/100
-
难度 2/5 1-3 小时 新手友好度 65/100
qgis/QGIS-Documentation#11275 ·
-
bug priority:normal ready-for-dev
难度 2/5 1-3 小时 新手友好度 88/100
OpenHands/extensions#626 · 1 条评论 ·
-
难度 1/5 1 小时以内 新手友好度 90/100
CSCfi/sd-search-api#39 ·
-
难度 1/5 1 小时以内 新手友好度 90/100