common-workflow-language / common-workflow-language/cwltool
anonymous directory are allowed to only have a listing
- Vorherrschende Sprache
- Python
- Sterne
- 376
- Forks
- 255
- Ø Merge
- 2 T. 7 Std.
- Gemergte PRs (30 T.)
- 12
Beschreibung
According to my reading of http://www.commonwl.org/v1.0/Workflow.html#Directory
```
if d["class"] == "Directory" and ("listing" not in d or "basename" not in d):
raise validate.ValidationException("Anonymous directory object must have 'listing' and 'basename' fields.")
```
is incorrect
https://github.com/common-workflow-language/cwltool/blob/master/cwltool/pathmapper.py#L59
@tetron Am I reading the spec wrong? Seems that if there is a listing and nothing else then "[t]he implementation must assign a unique identifier for the location field.", then the basename follows "If not provided, the implementation must set [the basename] based on the location field by taking the final path component after parsing location as an IRI."
This came up while debugging https://www.biostars.org/p/216220/ -- so perhaps the issue is that the reference implementation thinks this directory is anonymous when in fact it will have a real path in the output directory.
Beitragsleitfaden
Bewertung
Dieses Issue wurde noch nicht bewertet.