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

Restructure the docs #21

Merged
merged 8 commits into from
Nov 3, 2023
Merged

Restructure the docs #21

merged 8 commits into from
Nov 3, 2023

Conversation

wxicu
Copy link
Collaborator

@wxicu wxicu commented Oct 19, 2023

The docs should now have the following structure:

  • general
  • genetic demultiplexing
  • hashing demultiplexing
  • combining results: rescue mode

Copy link
Collaborator

@bio-la bio-la left a comment

Choose a reason for hiding this comment

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

the structure seems fine, we now need to have tutorials in place for the scenarios described

docs/source/general.md Show resolved Hide resolved
@bio-la
Copy link
Collaborator

bio-la commented Oct 20, 2023

some other quick things, it would be good to add figures we have already:

  • can we add the pipeline figure on the "general" landing page?
  • can we add the panels for genetic demultiplexing in the genetics section, similarly for the hashing, and the rescue mode figure?

@wxicu wxicu merged commit 80c611e into main Nov 3, 2023
4 checks passed
docs/source/general.md Show resolved Hide resolved
docs/source/general.md Show resolved Hide resolved
docs/source/general.md Show resolved Hide resolved
| Barcode | multiseq_1 | htodemux_1 | ... |
|:---------: |:----------: |:----------: |:---: |
| ... | ... | ... | ... |
- `adata` folder: stores Anndata object with filtered scRNA-seq read counts and assignment of each deconvolution method if `params.generate_anndata` is `True`. Details see section "scverse compatibility" above.
Copy link
Member

Choose a reason for hiding this comment

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

Does the equivalent not exist for mudata?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

still in Mariana' branch, not merged yet. But i can add in the doc now


```bash
sh test_data/download_data.sh
nextflow run main.nf
Copy link
Member

Choose a reason for hiding this comment

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

Maybe specify conda or docker here or something? Can you resemble a test profile like that the nf-core pipelines have?

docs/source/rescue.md Show resolved Hide resolved

Folder `data_output` with:

- an Anndata object which contains the filtered scRNA-seq counts from `params.rna_matrix_filered` and the assignment of the best-matched method pair after donor matching
Copy link
Member

Choose a reason for hiding this comment

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

where mudata?


### Optinal output: scverse compatibility

Folder `data_output` with:
Copy link
Member

Choose a reason for hiding this comment

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

didn't you say above that the folder is called adata?

Copy link
Collaborator Author

@wxicu wxicu Nov 3, 2023

Choose a reason for hiding this comment

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

above? You mean probably in general.md right? In genetic or hashing deconvolution workflow, no matter which mode to use,AnnData object is saved in for example$mode/gene_demulti/genetic_summary/adata folder. And there should be also a folder for MuData equivalently. For the rescue mode if donor matching is wished, an extra AnnData object with matched donor identity is saved rescue/data_output folder together with MuData

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

so here I only mentioned the additional adata output in the rescue mode compared with the genetic/hashing mode

test_data/download_data.sh Show resolved Hide resolved
@@ -18,7 +20,7 @@ tar -xzvf refdata-cellranger-hg19-3.0.0.tar.gz
# Download common variants
wget --load-cookies /tmp/cookies.txt "https://docs.google.com/uc?export=download&confirm=$(wget --quiet --save-cookies /tmp/cookies.txt --keep-session-cookies --no-check-certificate 'https://docs.google.com/uc?export=download&id=1lw4T6d7uXsm9dt39ZtEwpuB2VTY3wK1y' -O- | sed -rn 's/.*confirm=([0-9A-Za-z_]+).*/\1\n/p')&id=1lw4T6d7uXsm9dt39ZtEwpuB2VTY3wK1y" -O common_variants_hg19.vcf && rm -rf /tmp/cookies.txt
wget https://master.dl.sourceforge.net/project/cellsnp/SNPlist/genome1K.phase3.SNP_AF5e2.chr1toX.hg19.vcf.gz
gunzip genome1K.phase3.SNP_AF5e2.chr1toX.hg19.vcf.gz
#gunzip genome1K.phase3.SNP_AF5e2.chr1toX.hg19.vcf.gz
Copy link
Member

Choose a reason for hiding this comment

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

Why?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Because cellsnp can use the compressed file directly, dont need to unzip it. will delete the comment

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.

3 participants