Skip to content
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

TypeError: unhashable type: 'list' #141

Open
mr-c opened this issue Nov 16, 2021 · 2 comments
Open

TypeError: unhashable type: 'list' #141

mr-c opened this issue Nov 16, 2021 · 2 comments

Comments

@mr-c
Copy link
Member

mr-c commented Nov 16, 2021

  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
Copy link
Collaborator

mvdbeek commented Nov 16, 2021

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

@mvdbeek
Copy link
Collaborator

mvdbeek commented Nov 16, 2021

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

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