check `dat` argument (and others) at the start of save_all_plots()
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
找到 save_all_plots()，检查它如何使用 dat 和 convert_output() 的输出结构。添加对预期结构的早期验证，然后检查无效输入是否会及时失败并给出有用的消息，而不是生成许多 ... failed to run. 消息。
由索引模型根据 Issue 内容生成。
描述
I accidentally passed an object to the dat input of save_all_plots() that wasn't the output from convert_output() and I got 100+ lines of messages about ... failed to run. Adding a check at the start to confirm that the dat input is of the appropriate structure would allow the function to fail faster and more informatively.
- 主要语言
- R
- 星标
- 20
- 派生
- 10
- 平均合并
- 3 天 9 小时
- 30 天内合并 PR
- 21
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
nmfs-ost/stockplotr 的其他 Issue
-
[bug] plot_spawning_potential_ratio() example throwing error可能已有人在做 @JZahner1 于 1 天前认领。 未关闭
nmfs-ost/stockplotr#444 · 3 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
Release Checklist - Monthly Update可能已有人在做 @Schiano-NOAA 于 1 天前认领。 未关闭
nmfs-ost/stockplotr#442 · 已指派 2 人 ·
维护者通常 1 天内回复
-
update SS3 tests可能已有人在做 @Schiano-NOAA 于 9 天前认领。 未关闭
难度 4/5 3-5 天 新手友好度 35/100
nmfs-ost/stockplotr#441 · 4 条评论 ·
维护者通常 1 天内回复
-
Brainstorm ways to automatically update `save_all_plots()`可能已有人在做 @sbreitbart-NOAA 于 8 天前认领。 未关闭
nmfs-ost/stockplotr#432 · 4 条评论 · 已指派 2 人 ·
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 15/100
nmfs-ost/stockplotr#422 ·
维护者通常 1 天内回复
查看 nmfs-ost/stockplotr 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 78/100
rfordatascience/tidytuesday#1098 ·
-
难度 1/5 1 小时以内 新手友好度 84/100
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 72/100
microsoft/ML-For-Beginners#1056 ·
-
难度 2/5 1-3 小时 新手友好度 78/100
atomashevic/transforEmotion#46 ·
维护者通常 1 天内回复
-
`pk.calc.time_above()` with lin-up/log-down silently drops segments that end at a negative value未关闭
难度 2/5 1-3 小时 新手友好度 79/100
维护者通常 1 天内回复