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

Update Python dependencies #14195

Merged
merged 2 commits into from
Jul 25, 2022

Conversation

galaxybot
Copy link
Contributor

Run make update-dependencies.

@nsoranzo

This comment was marked as resolved.

@nsoranzo nsoranzo force-pushed the dev_auto_update_dependencies branch 3 times, most recently from cca36e4 to 26c8e86 Compare June 27, 2022 12:17
@nsoranzo

This comment was marked as resolved.

@jmchilton

This comment was marked as resolved.

@galaxybot galaxybot force-pushed the dev_auto_update_dependencies branch 2 times, most recently from b16ab93 to 949a57c Compare July 9, 2022 03:35
@galaxybot galaxybot force-pushed the dev_auto_update_dependencies branch from 949a57c to c9dc76d Compare July 16, 2022 03:39
@nsoranzo
Copy link
Member

The new Markdown 3.4.0 package broke sphinx-markdown-tables, so we need either a new release of the latter containing ryanfox/sphinx-markdown-tables#37 or to pin the former to <3.4 .

@galaxybot galaxybot force-pushed the dev_auto_update_dependencies branch from c9dc76d to 0efbb71 Compare July 23, 2022 03:40
galaxybot and others added 2 commits July 25, 2022 12:47
Fix the following traceback when running
`pytest test/unit/tool_util/test_cwl.py`:

```
    def test_boolean_defaults():
>       proxy = workflow_proxy(_cwl_tool_path("v1.2/tests/conditionals/cond-wf-002_nojs.cwl"))

test/unit/tool_util/test_cwl.py:244:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
lib/galaxy/tool_util/cwl/parser.py:760: in workflow_proxy
    return _to_cwl_workflow_object(workflow_path, strict_cwl_validation=strict_cwl_validation)
lib/galaxy/tool_util/cwl/parser.py:851: in _to_cwl_workflow_object
    cwl_workflow = _schema_loader(strict_cwl_validation).tool(path=workflow_path)
lib/galaxy/tool_util/cwl/schema.py:86: in tool
    process_definition = self.process_definition(raw_process_reference)
lib/galaxy/tool_util/cwl/schema.py:68: in process_definition
    loading_context, uri = resolve_and_validate_document(
.venv/lib/python3.8/site-packages/cwltool/load_tool.py:384: in resolve_and_validate_document
    document_loader.resolve_all(workflowobj, fileuri)
schema_salad/ref_resolver.py:965: in resolve_all
    ???
schema_salad/ref_resolver.py:1197: in validate_links
    ???
schema_salad/ref_resolver.py:1124: in validate_links
    ???
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

>   ???
E   schema_salad.exceptions.ValidationException: test/functional/tools/cwl_tools/v1.2/tests/conditionals/cond-wf-002_nojs.cwl:2:1: Field `cwlVersion`
E                                                                                     contains undefined
E                                                                                     reference to
E                                                                                     `file:///usr/users/ga002/soranzon/software/nsoranzo_galaxy/test/functional/tools/cwl_tools/v1.2/tests/conditionals/v1.2`

schema_salad/ref_resolver.py:1062: ValidationException
```

Note that the unit test doesn't fail if run on its own with
`pytest test/unit/tool_util/test_cwl.py::test_boolean_defaults` .
@nsoranzo nsoranzo force-pushed the dev_auto_update_dependencies branch from 0efbb71 to 09c0ed0 Compare July 25, 2022 12:16
@nsoranzo
Copy link
Member

Green at last! Can someone please merge?

@jdavcs jdavcs merged commit 34afba3 into galaxyproject:dev Jul 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants