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

multimodality #31

Open
wangjiawen2013 opened this issue Jul 12, 2019 · 3 comments
Open

multimodality #31

wangjiawen2013 opened this issue Jul 12, 2019 · 3 comments

Comments

@wangjiawen2013
Copy link

wangjiawen2013 commented Jul 12, 2019

Glad to be the first one here!
Looking forward to the final release version of epiScanpy and it's full tutorial and the download link of test data.
By the way, Could epiScanpy be used as a multi-modal analysis tool as discussed in scverse/scanpy#479 ?

@wangjiawen2013
Copy link
Author

I am following the tutorial now, when I run:
epi.ct.bld_atac_mtx(list_bam_files=list_cells,
loaded_feat=enhancers,
output_file='test_ATAC_mtx.txt',
path=path_to_play_data,
writing_option='w',
header=enhancer_names)

an error occurs:
pysam/libcalignmentfile.pyx in pysam.libcalignmentfile.IteratorRowAll.next()
OSError: truncated file
perhaps it is caused by samtools or pysam's version

Besides, the default value of parameter "chromosome" of function epi.ct.bld_atac_mtx is mouse genome, while that value is human genome in function epi.ct.load_features() is human. Why don't keep them the same ?

@wangjiawen2013
Copy link
Author

And the title of epiScanpy homepage is wrong, it should be epiScanpy, not eoiScanpy !
https://episcanpy.readthedocs.io/en/latest/

@DaneseAnna
Copy link
Collaborator

Sorry, for the delayed answer.

  • We have been working into making episcanpy a true multi-modal analysis tool. This is not fully implemented yet but I will post an update regarding this point soon.
  • Due to compatibility problems as well as other issues encountered with pysam, we switched to bamnostic in a later version of episcanpy.
  • We arel making sure to put the same default value of parameter "chromosome" in the later version of episcanpy
  • Episcanpy homepage have been updated to remove typos. Thanks, I was completely oblivious of this huge typo.

Thank you for all your remarks, this is very helpful!

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