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

adds synteny info to GSTF prep tool #148

Open
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

anilthanki
Copy link
Member

No description provided.

@nsoranzo
Copy link
Member

nsoranzo commented Jun 9, 2021

fixes travis errors

We have replaced TravisCI with GitHub workflows for a while 😉

tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
Anil Thanki and others added 12 commits June 10, 2021 13:04
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
cur.execute('SELECT gene_id FROM gene where species=? AND seq_region_name=? ORDER BY seq_region_start ASC',
(genome, ref, ))

return cur.fetchall()
Copy link
Member

Choose a reason for hiding this comment

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

That's probably a bit too far :D

I'd take lines 542-549 and make a function with them (you could re-use populate_synteny as a name) and merge the other 4 functions (fetch_genomes, fetch_seq_region_names, fetch_genes_by_order, populate_synteny) in it.

Anil Thanki and others added 2 commits June 11, 2021 09:45
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
tools/gstf_preparation/gstf_preparation.py Outdated Show resolved Hide resolved
Anil Thanki and others added 7 commits June 11, 2021 11:05
Anil Thanki and others added 3 commits June 15, 2021 08:57
…f_preparation/gstf_preparation.py

Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
…on.py

Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
…o fetch gene_id in tools/gstf_preparation/gstf_preparation.py

Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
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