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

handle the stop codons, COSMIC errors download, ENSEMBL releases #31

Merged
merged 44 commits into from
Feb 3, 2021

Conversation

ypriverol
Copy link
Member

@ypriverol ypriverol commented Jan 26, 2021

  • handle the stop codons in ensembl databases
  • add new test for stop codons.
  • Cosmic error handling when the user can't log in.
  • Ensembl release issue Ensembl release #23
  • Add changelog.md to the project
  • Add code quality service.

@ypriverol
Copy link
Member Author

@husensofteng do not merge yet the PR, still fixing COSMIC login error

@ypriverol ypriverol linked an issue Jan 26, 2021 that may be closed by this pull request
@ypriverol ypriverol mentioned this pull request Jan 26, 2021
3 tasks
@ypriverol
Copy link
Member Author

@husensofteng the ensembl grch37 is implemented. Please Review the present PR.

@ypriverol ypriverol changed the title handle the stop codons handle the stop codons, COSMIC errors download, ENSEMBL releases Jan 27, 2021
Squashed commits:
[c9fdd9a] code clean step
[78d4e5d] code clean step
[a31dc39] code clean step
[7895bd8] code clean step
[48ab4e7] change command config path
[de40c74] change command config path
[3a2da7b] still there
[376790e] minor examples
[fa84cdf] change command config path
[5effa00] change command config path
[010fbce] coverage updated
[2917989] change command config path
[e772df8] added unit tests
[c9dfe7d] code coverage
@ypriverol
Copy link
Member Author

ypriverol commented Jan 27, 2021

@husensofteng the PR is now to be reviewed.

@click.option('--config_file', '-c', help='Configuration to perform conversion between ENSEMBL Files',
default= this_dir + '../config/ensembl_config.yaml')
@click.option('-c', '--config_file', help='Configuration to perform conversion between ENSEMBL Files',
default= this_dir + '/../config/ensembl_config.yaml')
Copy link
Member

Choose a reason for hiding this comment

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

using this_dir makes issues when the default is used. why not just the default config file without this_dir?

pypgatk/commands/ensembl_downloader.py Show resolved Hide resolved
pypgatk/ensembl/data_downloader.py Show resolved Hide resolved
@ypriverol ypriverol merged commit f59ee0e into master Feb 3, 2021
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.

Ensembl release Add a changelog.md to the project Create a Nextflow workflow to create custom databases
2 participants