-
Notifications
You must be signed in to change notification settings - Fork 606
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
Oops! The 'qorts' MultiQC module broke... #821
Comments
Hi @nickluscombe, Sorry it's taken me so long to get to this. Now I'm here, I'm pretty confused though. When I run MultiQC with your file, it works fine without any errors. Looking at the code history, your error looks basically identical to #639 - this was fixed in 62fe089 back in December, which went out in MultiQC v1.4. Your log says that you're running MultiQC 1.6, so it should be fine if it's the same problem. How do you have MultiQC installed? Is there something strange going on with your installation? My first port of call would be to try reinstalling. Phil |
Closing as I cannot reproduce the error. |
@ewels I've bumped into the same issue. The problem seem to be the |
Thanks @jkokosar - do you have a file I can reproduce this with? I don't get the same error, even with basically the exact command used above:
|
@ewels you can use the file already attached to the issue - just add it to a subfolder, one level down from where you make a multiqc call.
|
Ahhh, I didn't have it in a subfolder - thanks! 👍 |
Hi all, This should now work in the latest v1.8dev of MultiQC. If you could give it a spin to confirm that would be great.. Thanks! Phil |
Works for me! Thank you! |
Hi Phil, I checked via
Best, |
Dear all, here is the trace that follows the error
As I understand the Let me know if this is not the right place to post this issue. Thanks, |
(bioinfo)
QC.summary.txt
The text was updated successfully, but these errors were encountered: