search

common-workflow-lab / galaxy

Fork of Galaxy (http://galaxyproject.org/) attempting to implement the CWL spec.
https://www.commonwl.org
Other
10 stars 2 forks source link

TypeError: unhashable type: 'list' #141

Open mr-c opened 2 years ago

mr-c commented 2 years ago 
  File "/home/michael/src/galaxy-cwl/lib/galaxy/tools/__init__.py", line 1780, in handle_single_execution
    rval = self.execute(
  File "/home/michael/src/galaxy-cwl/lib/galaxy/tools/__init__.py", line 1878, in execute
    return self.tool_action.execute(self, trans, incoming=incoming, set_output_hid=set_output_hid, history=history, **kwargs)
  File "/home/michael/src/galaxy-cwl/lib/galaxy/tools/actions/__init__.py", line 371, in execute
    params=wrapped_params.params,
  File "/home/michael/src/galaxy-cwl/lib/galaxy/tools/parameters/wrapped.py", line 45, in params
    self.wrap_values(self.tool.inputs, params, skip_missing_values=not self.tool.check_values)
  File "/home/michael/src/galaxy-cwl/lib/galaxy/tools/parameters/wrapped.py", line 89, in wrap_values
    element_identifier = element_identifier_mapper.identifier(value, input_values)
  File "/home/michael/src/galaxy-cwl/lib/galaxy/tools/wrappers.py", line 794, in identifier
    identifier_key = self.identifier_key_dict.get(dataset_value, None)
TypeError: unhashable type: 'list'
mvdbeek commented 2 years ago

test in question is CwlToolsTestCase.test_index1. We're expecting a single dataset as dataset_value, but getting a list here.

mvdbeek commented 2 years ago

I suppose this worked at one point, since it was in the PR description

Secondary files are supported at least partially, see the index1 and showindex1 CWL tools created to verify this as well as the test_index1 test case.

might be a recent change on the Galaxy side