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

Outputs metadata #135

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Outputs metadata #135

wants to merge 3 commits into from

Conversation

stevekm
Copy link
Member

@stevekm stevekm commented Feb 11, 2020

Changes made here in order to pass metadata out of the Roslin operator and set the output_metadata field for the job serializer in order to use the output metadata downstream in other operators

@stevekm stevekm changed the base branch from master to develop February 11, 2020 18:01
job_metadata['assay'] = assay
job_metadata['request_id'] = project_id
job_metadata['tumor'] = {}
job_metadata['tumor']['igo_id'] = tumor['igo_id']
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@allanbolipata @sivkovic which identifier are we using inside the Roslin pipelines? IGO ID or Patient ID or some other ID?

@stevekm
Copy link
Member Author

stevekm commented Feb 11, 2020

Once the inclusion of output metadata in the operator is complete, we will have to make sure to check against any unintended side effects in the behavior of the operators and job submission, etc.. Need some integration testing.

@stevekm
Copy link
Member Author

stevekm commented Feb 16, 2020

note: @nikhil is planning updates to Roslin CWL output schema, changes might need to be included here

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

Successfully merging this pull request may close these issues.

1 participant