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

add code to generate input.jsons for steps defined in workflows-mt.yaml #230

Open
aclum opened this issue Aug 6, 2024 · 5 comments
Open
Assignees
Labels

Comments

@aclum
Copy link
Contributor

aclum commented Aug 6, 2024

Depends on #222

logic will need to be added to decide what workflow component to run for nmdc:MetatranscriptomeExpressionAnalysis.

nmdc:MetatranscriptomeExpressionAnalysis is a new class which doesn't have any existing scheduler logic. The code use all docs (being used by the ref integrity squad) or an aggregation will need to traverse from omics_processing_set to library_preparation_set records to determine which of the following to run
Expression Analysis Antisense - if library_preparation_set.stranded_orientation ='antisense orientation'
Expression Analysis Sense - if library_preparation_set.stranded_orientation ='sense orientation'
Expression Analysis Nonstranded -if is_stranded = false
else run Expression Analysis Sense (default for JGI libraries)

@ssarrafan
Copy link

#222 is closed. Can this issue be completed today @mbthornton-lbl?

@aclum
Copy link
Contributor Author

aclum commented Aug 23, 2024

No, Michael's on vacation. Moving this to the next sprint.

@ssarrafan
Copy link

Two more weeks have gone by... any update on this issue @mbthornton-lbl

@aclum
Copy link
Contributor Author

aclum commented Sep 10, 2024

I'm going to backlog this. The MVP for metatranscriptomes is to use the workflows config, where only one Expression workflow is enabled at once, to set up the correct inputs.json instead of using this logic.

@aclum aclum added the backlog label Sep 10, 2024
@ssarrafan
Copy link

Removing from sprint and adding backlog label again
FYI @mbthornton-lbl @aclum

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants