Skip to content

Fixes #44: add popeve to dot_plot_v2.py#45

Open
lauragjensen wants to merge 4 commits into
mainfrom
fix44_popeve_dotplot
Open

Fixes #44: add popeve to dot_plot_v2.py#45
lauragjensen wants to merge 4 commits into
mainfrom
fix44_popeve_dotplot

Conversation

@lauragjensen
Copy link
Copy Markdown

  • Added popEVE to dot_plot_v2.py, including the -pltP plotting flag and -vep popeve filtering option.
  • Updated the README with documentation for popEVE.

Copy link
Copy Markdown
Contributor

@mtiberti mtiberti left a comment

Choose a reason for hiding this comment

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

can you add an example for v2 that contains popEVE? Similarly structured as the other examples

Comment thread dot_plot/README.md
- `-m/--mutations`: comma-separated mutations to display (e.g. `A4G,F55K`). Mutually exclusive with `-r`.
- `-r/--residues`: comma-separated residue positions to display (e.g. `4,55`). Mutually exclusive with `-m`.
- `-R/--revel_threshold`: REVEL pathogenic threshold (default: `0.5`).
- `-P/--popeve_threshold`: popEVE deleterious threshold (default: `-4.617`).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can you add somewhere in the docs that popeve is only available in v2?

@lauragjensen
Copy link
Copy Markdown
Author

Examples added and documentation updated.

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