[bug]: plot_abundance_at_age allows selection of non age-based modules
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 40/100
- Issue 类型
- 缺陷
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
- 技术栈
- r
调研方向
Start with plot_abundance_at_age and the filter_data step it calls, and see how the module is picked from the shared "abundance" label. Reproduce the error by selecting NUMBERS_AT_LENGTH or SPR_SERIES. Done means the function only offers modules that work with age-based plots, or it fails with a clear message. The maintainer has not chosen a fix, so confirm the approach in the thread before coding.
由索引模型根据 Issue 内容生成。
描述
The plot_abundance_at_age function currently identifies data based on the "abundance" label. However, many models reuse this label across modules, including "NUMBERS_AT_AGE", "NUMBERS_AT_LENGTH", "SPR_SERIES", "TIME_SERIES", etc. The default behavior of filter_data allows for user selection of which module to use for plotting, but some of these modules will throw errors if selected (e.g., "NUMBERS_AT_LENGTH").
This seems like it could be readily confusing for many users. It would probably be a good idea to restrict the allowable modules for users to select to only those that are applicable to the plotting function being used ("NUMBERS_AT_AGE" in this case).
This issue likely applies to more cases than the one identified here, and I am not confident as to a solution, but wanted to point it out.
- 主要语言
- R
- 星标
- 20
- 派生
- 8
- 平均合并
- 23 小时 33 分钟
- 30 天内合并 PR
- 9
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
nmfs-ost/stockplotr 的其他 Issue
-
[tests]: Set modules in all plot_* calls within test functions可能已有人在做 @Schiano-NOAA 于 1 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 68/100
nmfs-ost/stockplotr#416 · 5 条评论 · 1 个 reaction · 已指派 2 人 ·
维护者通常 1 天内回复
-
[tests]: are "rda file made when indicated" tests needed for every plot?可能已有人在做 @Schiano-NOAA 今天认领。 未关闭question
nmfs-ost/stockplotr#418 · 已指派 3 人 ·
维护者通常 1 天内回复
-
[feat[: add citation file: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-citation-files可能已有人在做 @sbreitbart-NOAA 于 1 天前认领。 未关闭
nmfs-ost/stockplotr#414 · 已指派 1 人 ·
维护者通常 1 天内回复
-
[feat]: Remove `html_all_figs_tables()` in favor of dashboard可能已有人在做 @Schiano-NOAA 于 1 天前认领。 未关闭
nmfs-ost/stockplotr#413 · 已指派 1 人 ·
维护者通常 1 天内回复
-
[feat]: Create function to help user directly add alt text and caption to csv for external objects可能已有人在做 @sbreitbart-NOAA 于 1 天前认领。 未关闭
nmfs-ost/stockplotr#412 · 已指派 1 人 ·
维护者通常 1 天内回复
查看 nmfs-ost/stockplotr 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 78/100
-
难度 2/5 1-3 小时 新手友好度 70/100
维护者通常 1 天内回复
-
enhancement
难度 2/5 1-3 小时 新手友好度 65/100
-
enhancement
难度 2/5 1-3 小时 新手友好度 70/100
insightsengineering/teal.modules.general#1022 · 1 个 reaction ·
-
enhancement
难度 2/5 1-3 小时 新手友好度 65/100