Addition of metadata generation for planemo run - #1434
Closed
Delphine-L wants to merge 9 commits into
Closed
Delphine-L wants to merge 9 commits into
Delphine-L wants to merge 9 commits into
Conversation
Delphine-L
marked this pull request as draft
March 5, 2024 18:17
Delphine-L
marked this pull request as ready for review
March 6, 2024 15:11
Collaborator
|
Would it be possible to add a test? Maybe here: https://github.com/galaxyproject/planemo/blob/5a0eb3185a02f71f6312b680b67e8bf0deae7b8d/tests/test_run.py |
Member
|
I think either |
Delphine-L
marked this pull request as draft
April 24, 2024 23:39
Contributor
Author
You're right. The info is in --test_output_json. Thank you ! |
Merge latest master to get fresh CI results on this long-open PR. Three adjacent-insertion conflicts, all resolved by keeping both sides: - galaxy/activity.py: master added 'import contextlib', branch added 'import json'; kept both in isort order. - options.py: master added run_use_cache_option, cwltool_cache_directory_option and test_use_cache_option next to this branch's run_output_metadata_option. - tests/test_run.py: master added three tests where this branch added test_run_metadata; the pre-existing PLANEMO_SKIP_GALAXY_TESTS decorator stays with test_run_download_output as on master. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Member
Closing this out - thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add the command
--output_metadata FILEto planemo run. Generate a json file containing the workflow ID, invocation ID, and history ID created by the 'run' command.