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

odgi source confusion #74

Open
anshumanmohan opened this issue Apr 27, 2023 · 2 comments
Open

odgi source confusion #74

anshumanmohan opened this issue Apr 27, 2023 · 2 comments
Assignees
Milestone

Comments

@anshumanmohan
Copy link
Contributor

anshumanmohan commented Apr 27, 2023

Or, whose odgi is it anyway

  1. slow-odgi does not strictly need odgi if you just want to run/benchmark it. It does need odgi if you want to run the tests, and in that case you need the built-from-source odgi because the source code contains fixes that were made on our request and those fixes are not yet available on Bioconda.
  2. exine needs odgi, and in particular it needs odgi's Python bindings. The most reliable way we know to do this is via Bioconda, which is why we suggest this route in the Pollen readme.

This works, but is a little gross. I just wanted to put this down somewhere so it's not just me who knows this dirty secret 😅

I'll poke around some to see if I can reliably solve this via one single installation of odgi.

@anshumanmohan
Copy link
Contributor Author

If #105 is okay-ed, we will no longer need the odgi Python bindings and this will go away for free.

@sampsyo
Copy link
Collaborator

sampsyo commented Jun 24, 2023

Excellent.

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