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

Np loom to h5ad snss2 #1363

Merged
merged 30 commits into from
Sep 12, 2024
Merged

Np loom to h5ad snss2 #1363

merged 30 commits into from
Sep 12, 2024

Conversation

nikellepetrillo
Copy link
Contributor

Description

Give your PR a concise yet descriptive title.
Please explain the changes you made here.
Explain the motivation for making this change. What existing problem does the pull request solve?
Mention any issues fixed, addressed, or otherwise related to this pull request, including issue numbers or hard links for issues in other repos.
You can delete these instructions once you have written your PR description.


Checklist

If you can answer "yes" to the following items, please add a checkmark next to the appropriate checklist item(s) and notify our WARP documentation team by tagging either @ekiernan or @kayleemathews in a comment on this PR.

  • Did you add inputs, outputs, or tasks to a workflow?
  • Did you modify, delete or move: file paths, file names, input names, output names, or task names?
  • If you made a changelog update, did you update the pipeline version number?

Copy link

Remember to squash merge!

Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
MultiSampleSmartSeq2SingleNucleus.wdl has not had its version updated
SmartSeq2SingleSample.wdl has not been changed and needs updating
MultiSampleSmartSeq2.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
MultiSampleSmartSeq2SingleNucleus.changelog.md has not been changed and needs to be updated
SmartSeq2SingleSample.changelog.md has not been changed and needs to be updated
MultiSampleSmartSeq2.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

Copy link

Remember to squash merge!

Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
MultiSampleSmartSeq2SingleNucleus.changelog.md has not been changed and needs to be updated
SmartSeq2SingleSample.changelog.md has not been changed and needs to be updated
MultiSampleSmartSeq2.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
MultiSampleSmartSeq2SingleNucleus.wdl has not had its version updated
SmartSeq2SingleSample.wdl has not been changed and needs updating
MultiSampleSmartSeq2.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

Copy link

Remember to squash merge!

Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
MultiSampleSmartSeq2SingleNucleus.changelog.md has not been changed and needs to be updated
SmartSeq2SingleSample.changelog.md has not been changed and needs to be updated
MultiSampleSmartSeq2.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
MultiSampleSmartSeq2SingleNucleus.wdl has not had its version updated
SmartSeq2SingleSample.wdl has not been changed and needs updating
MultiSampleSmartSeq2.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

Copy link

Remember to squash merge!

Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
MultiSampleSmartSeq2SingleNucleus.changelog.md has not been changed and needs to be updated
SmartSeq2SingleSample.changelog.md has not been changed and needs to be updated
MultiSampleSmartSeq2.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
MultiSampleSmartSeq2SingleNucleus.wdl has not had its version updated
SmartSeq2SingleSample.wdl has not been changed and needs updating
MultiSampleSmartSeq2.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

Copy link

Remember to squash merge!

Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
MultiSampleSmartSeq2SingleNucleus.changelog.md has not been changed and needs to be updated
SmartSeq2SingleSample.changelog.md has not been changed and needs to be updated
MultiSampleSmartSeq2.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
MultiSampleSmartSeq2SingleNucleus.wdl has not had its version updated
SmartSeq2SingleSample.wdl has not been changed and needs updating
MultiSampleSmartSeq2.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

Copy link

Remember to squash merge!

Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
MultiSampleSmartSeq2SingleNucleus.wdl has not had its version updated
SmartSeq2SingleSample.wdl has not been changed and needs updating
MultiSampleSmartSeq2.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

Copy link

github-actions bot commented Sep 6, 2024

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
MultiSampleSmartSeq2SingleNucleus.changelog.md has not been changed and needs to be updated
MultiSampleSmartSeq2.changelog.md has not been changed and needs to be updated
PairedTag.changelog.md has not been changed and needs to be updated
Optimus.changelog.md has not been changed and needs to be updated
atac.changelog.md has not been changed and needs to be updated
SmartSeq2SingleSample.changelog.md has not been changed and needs to be updated
Multiome.changelog.md has not been changed and needs to be updated
SlideSeq.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

Copy link

github-actions bot commented Sep 6, 2024

Remember to squash merge!

Copy link

github-actions bot commented Sep 6, 2024

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
MultiSampleSmartSeq2SingleNucleus.wdl has not had its version updated
MultiSampleSmartSeq2.wdl has not been changed and needs updating
PairedTag.wdl has not been changed and needs updating
Optimus.wdl has not had its version updated
atac.wdl has not been changed and needs updating
SmartSeq2SingleSample.wdl has not been changed and needs updating
Multiome.wdl has not been changed and needs updating
SlideSeq.wdl has not had its version updated
Some WDLs or changelog files need updating. See output for details.
validation_failed

Copy link

github-actions bot commented Sep 6, 2024

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
MultiSampleSmartSeq2SingleNucleus.changelog.md has not been changed and needs to be updated
MultiSampleSmartSeq2.changelog.md has not been changed and needs to be updated
PairedTag.changelog.md has not been changed and needs to be updated
Optimus.changelog.md has not been changed and needs to be updated
atac.changelog.md has not been changed and needs to be updated
SmartSeq2SingleSample.changelog.md has not been changed and needs to be updated
Multiome.changelog.md has not been changed and needs to be updated
SlideSeq.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

Copy link

github-actions bot commented Sep 6, 2024

Remember to squash merge!

Copy link

github-actions bot commented Sep 6, 2024

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
MultiSampleSmartSeq2SingleNucleus.changelog.md has not been changed and needs to be updated
MultiSampleSmartSeq2.changelog.md has not been changed and needs to be updated
PairedTag.changelog.md has not been changed and needs to be updated
Optimus.changelog.md has not been changed and needs to be updated
atac.changelog.md has not been changed and needs to be updated
SmartSeq2SingleSample.changelog.md has not been changed and needs to be updated
Multiome.changelog.md has not been changed and needs to be updated
SlideSeq.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

Copy link

github-actions bot commented Sep 6, 2024

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
MultiSampleSmartSeq2SingleNucleus.wdl has not had its version updated
MultiSampleSmartSeq2.wdl has not been changed and needs updating
PairedTag.wdl has not been changed and needs updating
Optimus.wdl has not had its version updated
atac.wdl has not been changed and needs updating
SmartSeq2SingleSample.wdl has not been changed and needs updating
Multiome.wdl has not been changed and needs updating
SlideSeq.wdl has not had its version updated
Some WDLs or changelog files need updating. See output for details.
validation_failed

@@ -297,98 +298,7 @@ task SingleNucleusOptimusLoomOutput {
}


task SingleNucleusSmartSeq2LoomOutput {
Copy link
Contributor

Choose a reason for hiding this comment

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

@nikellepetrillo Should we leave the original task in place? We left the other Loom tasks in.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ekiernan I think we can leave it so it is consistent with other pipelines. Maybe one day we can clean up all unused tasks.

Copy link
Contributor

@ekiernan ekiernan left a comment

Choose a reason for hiding this comment

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

Looks good! But I had a comment about whether we wanted to delete the old task. Thank you for all the work you did on this!

Copy link

Remember to squash merge!

Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
MultiSampleSmartSeq2SingleNucleus.changelog.md has not been changed and needs to be updated
MultiSampleSmartSeq2.changelog.md has not been changed and needs to be updated
PairedTag.changelog.md has not been changed and needs to be updated
Optimus.changelog.md has not been changed and needs to be updated
atac.changelog.md has not been changed and needs to be updated
SmartSeq2SingleSample.changelog.md has not been changed and needs to be updated
Multiome.changelog.md has not been changed and needs to be updated
SlideSeq.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
MultiSampleSmartSeq2SingleNucleus.wdl has not had its version updated
MultiSampleSmartSeq2.wdl has not been changed and needs updating
PairedTag.wdl has not been changed and needs updating
Optimus.wdl has not had its version updated
atac.wdl has not been changed and needs updating
SmartSeq2SingleSample.wdl has not been changed and needs updating
Multiome.wdl has not been changed and needs updating
SlideSeq.wdl has not had its version updated
Some WDLs or changelog files need updating. See output for details.
validation_failed

Copy link

Remember to squash merge!

Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
MultiSampleSmartSeq2.changelog.md has not been changed and needs to be updated
PairedTag.changelog.md has not been changed and needs to be updated
Optimus.changelog.md has not been changed and needs to be updated
atac.changelog.md has not been changed and needs to be updated
SmartSeq2SingleSample.changelog.md has not been changed and needs to be updated
Multiome.changelog.md has not been changed and needs to be updated
SlideSeq.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
MultiSampleSmartSeq2.wdl has not been changed and needs updating
PairedTag.wdl has not been changed and needs updating
Optimus.wdl has not had its version updated
atac.wdl has not been changed and needs updating
SmartSeq2SingleSample.wdl has not been changed and needs updating
Multiome.wdl has not been changed and needs updating
SlideSeq.wdl has not had its version updated
Some WDLs or changelog files need updating. See output for details.
validation_failed

Copy link

Remember to squash merge!

Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@nikellepetrillo nikellepetrillo merged commit 07f8b2f into develop Sep 12, 2024
5 of 6 checks passed
@nikellepetrillo nikellepetrillo deleted the np-loom-to-h5ad-snss2 branch September 12, 2024 12:09
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.

4 participants