Skip to content

Added anadama2 and WDL commands and tutorials#2

Open
sagun98 wants to merge 6 commits into
mainfrom
WDL/Anadama2
Open

Added anadama2 and WDL commands and tutorials#2
sagun98 wants to merge 6 commits into
mainfrom
WDL/Anadama2

Conversation

@sagun98
Copy link
Copy Markdown
Collaborator

@sagun98 sagun98 commented Jul 10, 2025

Hi @kescobo,
I added anadama2 and WDL commands and tutorials in this PR branch, however I still need to double check everything in the VM before tomorrow and take a snapshot if needed.

@sagun98 sagun98 mentioned this pull request Jul 10, 2025
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think you meant to include this, did you? It needs to be at the top level if so

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Agreed. I have deleted the .github directory in the new commit.

Comment thread anadama2/Readme.md Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Guessing this is identical to the anadama tutorial that already exists? We probably don't need to duplicate it here. If you want to include it - maybe do it as a submodule?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Agreed. This is a redundant README.md file. Removed it from the latest commit.

Comment thread WDL/wmgx.wdl Outdated
# create output dir if it doesn’t exist
mkdir -p "${output_dir}"

biobakery_workflows wmgx \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is just running the anadama-constructed workflow. I thought the idea was to show how we'd define the workflow using WDL (so eg having kneaddata, metaphlan, and humann steps)?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Agreed. I have updated the commit to include WDL workflow-kneaddata + metaphlan + humann steps rather than just the one biobakery_workflows command. I will test out the changes in the VM next.

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.

2 participants