Skip to content

Estimation motifs #78

Merged
merged 8 commits into from
Jan 19, 2019
Merged

Estimation motifs #78

merged 8 commits into from
Jan 19, 2019

Conversation

renewiegandt
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@HendrikSchultheis HendrikSchultheis left a comment

Choose a reason for hiding this comment

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

Fix the typos and we are good to go.

README.md Outdated

**Important Notes:**
1. For conda the channel bioconda needs to be set as highest priority! This is required due to two different packages with the same name in different channels. For the pipeline the package jellyfish from the channel bioconda is needed and **NOT** the jellyfish package from the channel conda-forge!
2. The paramtere --create_known_tfbs_path and --tfbs_path need the absolute path. If a relative path is given it will not work due to nextflow changing the working directory. This will soon be updated.
Copy link
Collaborator

Choose a reason for hiding this comment

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

...need an absolute path.

README.md Outdated
nextflow run pipeline.nf --bigwig ./demo/buenrostro50k_chr1_fp.bw --bed ./demo/buenrostro50k_chr1_peaks.bed --genome_fasta ./demo/hg38/hg38_chr1.fa --motif_db ./demo/motif_database/jaspar_vertebrates.meme --out ./demo/buenrostro50k_chr1_out/ --create_known_tfbs_path ./demo/known_tfbs_hg38_chr1/ --organism hg38
nextflow run pipeline.nf --bigwig ./demo/buenrostro50k_chr1_fp.bw --bed ./demo/buenrostro50k_chr1_peaks.bed --genome_fasta ./demo/hg38/hg38_chr1.fa --motif_db ./demo/motif_database/jaspar_vertebrates.meme --out ./demo/buenrostro50k_chr1_out/ --create_known_tfbs_path [absolute path]/demo/known_tfbs/ --organism hg38
```
**Important Note:** In can happen that tfbsscan does not work due to an unknown bug. If this is the case you will see the error message shown in "Known Issues". If this occurs and "fimo" will also throw this error, try following command:
Copy link
Collaborator

Choose a reason for hiding this comment

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

'It' not 'in'
...shown in the Known Issues section below.
If the error occurs first try to change the tfbsscan_method to fimo. If it persists try the following command:

README.md Outdated
```
**Important Note:** In can happen that tfbsscan does not work due to an unknown bug. If this is the case you will see the error message shown in "Known Issues". If this occurs and "fimo" will also throw this error, try following command:
```
nextflow run pipeline.nf --bigwig ./demo/buenrostro50k_chr1_fp.bw --bed ./demo/buenrostro50k_chr1_peaks.bed --genome_fasta ./demo/hg38/hg38_chr1.fa --motif_db ./demo/motif_database/jaspar_vertebrates.meme --out ./demo/buenrostro50k_chr1_out/ --tfbs_path [absolute path]/demo/known_tfbs_chr1_hg38/ --organism hg38
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please verify that this and the nextflow call above are correct. To me it seems that you just changed the parameter create_known_tfbs_path to tfbs_path between the two.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh I think I figured it out. The tfbs_scan provides already scanned data so tfbsscan can be skipped, right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, that's correct. With the parameter tfbs_path, you can provide scanned data. So tfbsscan.py will be skipped.

README.md Outdated
--create_known_tfbs_path Path to directory where output from tfbsscan (known motifs) are stored.
Path can be set as tfbs_path in next run. (Needs absolute path)
or
--tfbs_path Path to directory with output from tfbsscan. If given tfbsscan will not be run. (Needs absolute path)
Copy link
Collaborator

Choose a reason for hiding this comment

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

...will be skipped.

README.md Outdated
Moitf clustering:
--cluster_motif Boolean If 1 pipeline clusters motifs. If its 0 it does not. (Default: 0)
--cluster_motif Boolean If 1 pipeline clusters motifs. If its 0 it does not. (Defaul: 0)
Copy link
Collaborator

Choose a reason for hiding this comment

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

*Default

@renewiegandt renewiegandt merged commit 8be83da into dev Jan 19, 2019
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants