parsing CWL v1.0 & v1.1
还没有人认领这个 Issue。
评估
调研方向
从 Dockstore 的 CWLHandler.java 约第 364 行开始，使用两个已链接的主描述符复现 loadDocument。检查 issue 评论中的 ValidationException 消息和堆栈跟踪详细信息，然后跟踪对应的 cwljava 验证路径。完成的标准是识别并修复解析失败，并为两个 workflow 提供回归覆盖，或记录任一描述符无效的原因。
由索引模型根据 Issue 内容生成。
描述
Recently, we integrated cwljava v1.0 into Dockstore, and during testing, we found some workflows that cause the parser to throw. In our webservice, we use our own preprocessor to combine the various component CWL files into one big CWL represented by Maps/Lists, and parse it with cwljava here:
https://github.com/dockstore/dockstore/blob/67f4547e771864cafacdc1c92fa7bd47261e32da/dockstore-webservice/src/main/java/io/dockstore/webservice/languages/CWLHandler.java#L364
The following workflows cause loadDocument to throw a ValidationException:
https://github.com/ICGC-TCGA-PanCancer/OxoG-Dockstore-Tools/tree/master
primary descriptor: /oxog_varbam_annotate_wf.cwl
https://github.com/h3abionet/h3agatk/tree/1.0.1
primary descriptor: /workflows/GATK/GATK-complete-WES-Workflow-h3abionet.cwl
The first workflow contains a SchemaDefRequirement and the parser appears to have trouble parsing the type references (TumourType.yaml#TumourType etc). When I change the type references to int, the workflow successfully parses.
Judging from exception message, the second workflow seems to be failing for a different reason, but I haven't pinpointed what, exactly. It is possible that it's not valid, but a cursory inspection didn't turn up any problems.
The exception messages are pretty big, so I put them and some stack trace info in the comments.
Please let me know if you need any more info. Thanks!
- 主要语言
- Java
- 星标
- 16
- 派生
- 8
- 平均合并
- 3 小时 25 分钟
- 30 天内合并 PR
- 4
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
common-workflow-language/cwljava 的其他 Issue
-
难度 4/5 3-5 天 新手友好度 38/100
-
难度 5/5 一周以上 新手友好度 20/100
common-workflow-language/cwljava#96 · 5 条评论 ·
-
难度 5/5 一周以上 新手友好度 25/100
common-workflow-language/cwljava#93 · 1 个 reaction ·
-
enhancement help wanted
难度 5/5 一周以上 新手友好度 25/100
-
难度 4/5 3-5 天 新手友好度 20/100
common-workflow-language/cwljava#43 · 4 条评论 ·
查看 common-workflow-language/cwljava 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 76/100
apache/rocketmq-dashboard#7250 ·
维护者通常 3 天内回复
-
HttpMaxFrameSizeTest.sendOversizedMessageTest() passes silently when maxFrameSize is not enforced未关闭
难度 1/5 1 小时以内 新手友好度 90/100
apache/activemq#2706 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
federicoiosue/Omni-Notes#1032 ·
-
难度 2/5 1-3 小时 新手友好度 78/100
unitycatalog/unitycatalog#2025 ·
维护者通常 4 天内回复
-
难度 1/5 1-3 小时 新手友好度 78/100
krizaka/krizaka-notifications#22 ·
维护者通常 1 天内回复