Skip to content

KeyError: 'location' - Cannot make job: 'location' #303

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
gijzelaerr opened this issue Feb 24, 2017 · 4 comments
Open

KeyError: 'location' - Cannot make job: 'location' #303

gijzelaerr opened this issue Feb 24, 2017 · 4 comments

Comments

@gijzelaerr
Copy link
Member

Looks like i found an other bug while writing these CWL files:

https://github.com/gijzelaerr/vermeerkat-cwl/tree/master/cwl

.virtualenv/bin/cwl-runner --outdir results --cachedir cache cwl/workflow.cwl cwl/job.cwl
.virtualenv/bin/cwl-runner 1.0.20170223161812
Resolved 'cwl/workflow.cwl' to 'file:///home/gijs/Work/vermeerkat-cwl/cwl/workflow.cwl'
[job curl] Using cached output in /home/gijs/Work/vermeerkat-cwl/cache/175791fc94a768e7e925838df7b083b2
[step curl] completed success
[job h5toms] Using cached output in /home/gijs/Work/vermeerkat-cwl/cache/106bf4457e4f2930c4189f9e7158d0ed
[step h5toms] completed success
Unexpected exception
Traceback (most recent call last):
  File "/home/gijs/Work/cwltool/cwltool/workflow.py", line 587, in job
    **kwargs):
  File "/home/gijs/Work/cwltool/cwltool/draft2tool.py", line 190, in job
    separateDirs=False)
  File "/home/gijs/Work/cwltool/cwltool/pathmapper.py", line 150, in __init__
    self.setup(dedup(referenced_files), basedir)
  File "/home/gijs/Work/cwltool/cwltool/pathmapper.py", line 104, in dedup
    if r["location"] not in marksub and r["location"] not in markdup:
  File "/home/gijs/Work/vermeerkat-cwl/.virtualenv/local/lib/python2.7/site-packages/ruamel/yaml/comments.py", line 595, in __getitem__
    return ordereddict.__getitem__(self, key)
KeyError: 'location'
[step rfimasker] Cannot make job: 'location'
@gijzelaerr
Copy link
Member Author

If I manually set the 'location' field in the job file, It complains I need to set basename also. If I also set that field it runs, but the variable field is empty.

@gijzelaerr
Copy link
Member Author

AHA! I accidentally set the field name to file, not File.

@mr-c
Copy link
Member

mr-c commented Feb 27, 2017

@gijzelaerr :-) All good now?

Can you link to the commit that gave you the above error? I would still like to improve cwltool's error reporting

@gijzelaerr
Copy link
Member Author

yeah it is working, but the error is a bit confusing.

Sorry, forgot to link to the commit directly:

https://github.com/gijzelaerr/vermeerkat-cwl/blob/5ce4b56269d4a854e9522fb17bd76eedc9a4edd2/cwl/job.cwl

tetron pushed a commit that referenced this issue Jul 23, 2017
…n-docs

put stdin shortcut in correct place
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants