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

Update ChIP-Seq tutorial with latest tool versions and links #4290

Merged
merged 3 commits into from
Jul 17, 2023

Conversation

pavanvidem
Copy link
Member

No description provided.

@pavanvidem pavanvidem requested a review from a team as a code owner July 17, 2023 12:34
@@ -306,7 +307,7 @@ Since in this tutorial we are interested in assessing H3K4me3, H3K27me3 and CTCF
> ```
>
> 3. Rename the files
> 3. **multiBamSummary** {% icon tool %} with the following parameters
> 3. {% tool [multiBamSummary](toolshed.g2.bx.psu.edu/repos/bgruening/deeptools_multi_bam_summary/deeptools_multi_bam_summary/3.5.1.0.0) %} with the following parameters:
Copy link
Member

@anuprulez anuprulez Jul 17, 2023

Choose a reason for hiding this comment

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

Can we use the latest version of the multiBamSummary tool which is: Galaxy Version 3.5.2+galaxy0 or is there a specific reason to use one prior version?

Copy link
Member Author

Choose a reason for hiding this comment

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

because this is the most common latest version on .org and .eu

@@ -322,7 +323,7 @@ Since in this tutorial we are interested in assessing H3K4me3, H3K27me3 and CTCF
>
> Using these parameters, the tool will take bins of 1000 bp separated by 500 bp on the chromosome X. For each bin the overlapping reads in each sample will be computed and stored into a matrix.
>
> 4. **plotCorrelation** {% icon tool %} with the following parameters
> 4. {% tool [plotCorrelation](toolshed.g2.bx.psu.edu/repos/bgruening/deeptools_plot_correlation/deeptools_plot_correlation/3.5.1.0.0) %} with the following parameters:
Copy link
Member

Choose a reason for hiding this comment

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

Similar comment as previously made for the multiBamSummary tool, can we use the latest version?

@anuprulez
Copy link
Member

I ran this tutorial on EU, and it works fine. I verified all the updated steps except for the very last plot that compares all the 8 pairs of samples (which is time-consuming).

history: https://usegalaxy.eu/u/kumara/h/chip-seq-1

ping @pavanvidem

@anuprulez anuprulez enabled auto-merge July 17, 2023 16:22
@anuprulez anuprulez merged commit b1277bc into galaxyproject:main Jul 17, 2023
3 checks passed
@pavanvidem
Copy link
Member Author

thanks @anuprulez @gallardoalba

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

Successfully merging this pull request may close these issues.

3 participants