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

Add methylation analysis methods #21

Merged
merged 12 commits into from
Jun 13, 2023
Merged

Add methylation analysis methods #21

merged 12 commits into from
Jun 13, 2023

Conversation

ewafula
Copy link
Contributor

@ewafula ewafula commented Jun 8, 2023

Purpose

Add methylation analysis methods

  • Methylation array preprocessing
  • Methylation beta-values summaries
  • Methylation sample classification

Issue

Directions for reviewers. Tell potential reviewers what kind of feedback you are soliciting.

Which areas should receive a particularly close look?

Method description

Is there anything that you want to discuss further?

PR associated with OpenPedCan-analysis PR # 378 which requires review and merging

Is the pull request ready for review?

Yes

Pull review checklist

Unless otherwise noted above, this PR will be considered ready for review when all four items have been checked.

Spellcheck Step

The dictionary used for spellchecking can be updated.
Edit the file in build/assets/custom-dictionary.txt by adding new entries to the end.
You do not need to change anything else.
However, if you want to update the first line to have an accurate count of words and you want to remove non-unique ones, run the following command from within build/assets on your favorite OS X or Linux machine:

(( len = $(awk '!a[$0]++' < custom-dictionary.txt | wc -l ) - 1 )); tmpfile="$(mktemp)"; echo "personal_ws-1.1 en $len utf-8" > $tmpfile; tail -n +2 custom-dictionary.txt | awk '!a[$0]++' >> $tmpfile; mv $tmpfile custom-dictionary.txt

Copy link
Member

@jharenza jharenza left a comment

Choose a reason for hiding this comment

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

thanks so much @ewafula - made some suggestions and had a question

content/06.methods.md Outdated Show resolved Hide resolved
content/06.methods.md Outdated Show resolved Hide resolved
content/06.methods.md Outdated Show resolved Hide resolved
content/06.methods.md Outdated Show resolved Hide resolved
content/06.methods.md Outdated Show resolved Hide resolved
content/06.methods.md Outdated Show resolved Hide resolved
content/06.methods.md Outdated Show resolved Hide resolved
content/06.methods.md Outdated Show resolved Hide resolved
ewafula and others added 8 commits June 12, 2023 09:52
Co-authored-by: Jo Lynne Rokita <jharenza@gmail.com>
Co-authored-by: Jo Lynne Rokita <jharenza@gmail.com>
Co-authored-by: Jo Lynne Rokita <jharenza@gmail.com>
Co-authored-by: Jo Lynne Rokita <jharenza@gmail.com>
Co-authored-by: Jo Lynne Rokita <jharenza@gmail.com>
Co-authored-by: Jo Lynne Rokita <jharenza@gmail.com>
Co-authored-by: Jo Lynne Rokita <jharenza@gmail.com>
content/06.methods.md Outdated Show resolved Hide resolved
Copy link
Member

@jharenza jharenza left a comment

Choose a reason for hiding this comment

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

approved once the final comment is addressed, thank you!

This pull request was closed.
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