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

shapeerror #47

Closed
wwood opened this issue Dec 1, 2023 · 7 comments · Fixed by #48
Closed

shapeerror #47

wwood opened this issue Dec 1, 2023 · 7 comments · Fixed by #48

Comments

@wwood
Copy link
Collaborator

wwood commented Dec 1, 2023

Running this from aviary (which didn't error out as probably should have)

[2023-12-01T19:27:11Z INFO  rosella] rosella version 0.5.1
[2023-12-01T19:27:11Z INFO  rosella::recover::recover_engine] Calculating contig coverages.
[2023-12-01T19:27:16Z INFO  rosella::recover::recover_engine] Calculating TNF table.
[2023-12-01T19:29:39Z ERROR rosella] Recover Failed with error: ShapeError/IncompatibleShape: incompatible shapes

What more info did you want?

@wwood
Copy link
Collaborator Author

wwood commented Dec 1, 2023

This also happened when refining from metabat2, perhaps predictably

@sjparkmk
Copy link

sjparkmk commented Dec 4, 2023

hi there, i have the same error. the contigs were prepared by megahit (v1.2.9).

@rhysnewell
Copy link
Owner

I think I have a solution to this if either of you is willing to share your assembly with me I can test whether it is fixed and push out a new release. Otherwise, potential fix is on branch #ISS-47

@sjparkmk
Copy link

sjparkmk commented Dec 4, 2023

I think I have a solution to this if either of you is willing to share your assembly with me I can test whether it is fixed and push out a new release. Otherwise, potential fix is on branch #ISS-47

thanks for your reply.. but how could i share the huge file (ca. 7.6 Gb) to you?

could i use coverM? (#39 (comment))

@rhysnewell
Copy link
Owner

Very true, that won't work. Are you willing to try and clone the branch I mentioned and install via cargo? Install instructions should outline how, otherwise I will continue trying to reproduce the error on my end

@sjparkmk
Copy link

sjparkmk commented Dec 4, 2023

Very true, that won't work. Are you willing to try and clone the branch I mentioned and install via cargo? Install instructions should outline how, otherwise I will continue trying to reproduce the error on my end

Thanks a lot. OK. i am going to install and attempt coverM. and.. could i install via bioconda? cargo is better (i meant stable for install) than conda? if not, i will use conda. cheers!

@rhysnewell
Copy link
Owner

I was able to reproduce the issue and have fixed in v0.5.2
This new version should be available via conda soon

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 a pull request may close this issue.

3 participants