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

Kernal crashing at get_pair_locations #69

Open
annrosebright opened this issue Jul 31, 2023 · 3 comments
Open

Kernal crashing at get_pair_locations #69

annrosebright opened this issue Jul 31, 2023 · 3 comments

Comments

@annrosebright
Copy link

Hi,

Thank you for generating this great tool. I am having an issue while running the following step:
pairs = Ce16.get_pair_locations(("TCF4_1","NFIB_1"))

The kernel is always crashing and I am not able to understand what the problem is. Could you suggest some way I could identify the issue or have a workaround to get the location of regions where certain pairs of TFs are present?

Thanks a lot in advance.
Best,
Ann

@msbentsen
Copy link
Member

Hi Ann,

Can you try it with the TF-COMB from the development branch by installing:
pip install git+https://github.com/loosolab/TF-COMB@dev

The version (1.0.4) is not released yet, but I believe we solved a bug related to this. I hope that solves the issue!

@annrosebright
Copy link
Author

Thanks a lot, it worked.

Another question: is there a way to manage the pdf size while using:
Ce16.plot_TFBS(window_chrom = "chr9",
window_start = 73935027,
window_end = 73935159,
fasta="/datastore_share/Users/ann/genome/GRCm38.p6.genome.fa",save = "Chr9_NFIB_TCF4.pdf", TFBS_track_height=6)

I can manage the track height but can i manage the width of the track or pdf as well?

Thank you.

@msbentsen
Copy link
Member

I can manage the track height but can i manage the width of the track or pdf as well?

Good question - I realized there was no method for this, so I just added the option "figsize" to the new version TF-COMB=1.1. There is an example here:
https://tf-comb.readthedocs.io/en/latest/examples/plot_TFBS_genome_view.html#Set-figsize-to-control-size-of-plot-(the-default-is-to-estimate-it-from-the-number-of-tracks)
which shows how to use it to control the final size of the figure, so then you can adjust until it is right.

So yes, if you just update the version, that should be available!

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

No branches or pull requests

2 participants