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

fusioncatcher references version #95

Closed
valleema opened this issue Jan 22, 2020 · 11 comments
Closed

fusioncatcher references version #95

valleema opened this issue Jan 22, 2020 · 11 comments
Assignees

Comments

@valleema
Copy link

Hello,

I am trying the pipeline and I ran into a little problem that prevents the use of fusioncatcher.

In the script the references are outdated.

So I locally modified it to download the v98 (the currently available one).

However, it does not work when the pipeline is launched.

Maybe due to what is bundled in the singularity image, this is the exit message from Nextflow execution:

  ....................
  ERROR: The version of the data build does not match the version of this pipeline!
  Please, run again the 'fusioncatcher-build.py' in order to fix this!
  ....................

What should I do to resolve it?

@maxulysse maxulysse self-assigned this Jan 22, 2020
@maxulysse
Copy link
Member

Hi @valleema I more recently got more involved into this project, and I'm currently focusing on the #54 issue.
I'll look into that first thing after.

@valleema
Copy link
Author

Great! And the #54 issue is quite interesting to add, I reckon.

@matq007
Copy link
Member

matq007 commented Jan 22, 2020

Hey @valleema are you running nf-core/rnafusion:1.0.2? Everything should be compatible

@valleema
Copy link
Author

Hi @matq007, yes I am running the "downloaded-yesterday". In the Nextflow code, the download points to a v90 reference, and only the v98 is available.

That's why I manually downloaded the v98, following your help pages.

@matq007
Copy link
Member

matq007 commented Jan 22, 2020

Ah, I finally see the problem. Damn.. the author removed the old reference. Hmmm... The only option currently that I can think about is to create your own singularity image with the latest fusioncatcher tool.

new environment file

name: fusioncatcher
channels:
  - bioconda
  - conda-forge
dependencies:
  - bioconda::fusioncatcher=1.20

and then cd tools && build.sh fusioncatcher which will generate docker container which can be afterwards transformed into singularity image.

@valleema
Copy link
Author

Yes, building my own image was on my mind as well... Tough task, right now as I am not root of any machines around. I might find a way, but it could be a better call to let you time to update the image, for the new users of this pipeline as well.

What do you think?

@valleema
Copy link
Author

Or... If you have them somewhere, make the v90 files available to download. It could be a quick solution.

@matq007
Copy link
Member

matq007 commented Jan 23, 2020

I have fixed the issue on the dev branch. There is a new container on dockerhub which you can download and convert to singularity image. One thing to note is that in order to run fusioncatcher you have to run fusioncatcher.py <commands>.

@matq007 matq007 closed this as completed Jan 23, 2020
@valleema
Copy link
Author

So, it needs to be 1.20 here :

fusioncatcher_version = '1.00'

fusioncatcher_version = '1.00'

fusioncatcher_version = '1.00'

But so far, so good, it runs! Thanks a lot!!!

@matq007
Copy link
Member

matq007 commented Jan 24, 2020

Ah, thanks for pointing out. We will make these changes once we bump the version

@valleema
Copy link
Author

Great. Thanks a lot for your help and your quick solutions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants