I updated MultiQC from version 1.27 to 1.28 in <#C...
# multiqc
c
I updated MultiQC from version 1.27 to 1.28 in #C06TRU74N4D and it messed one of the plot displays. Is that a know issue?
p
The fact that it says "Loading" in the sidebar on the right tells me that there is a JavaScript error that has crashed the report
That particular missing plot is probably a red herring - it just crashed before it got to that part of the report when rendering
Can you look in the browser developer JavaScript console please? There should be a nice big red error about something.
But yes this sounds very much like it's a bug that we will need to fix. This basically should never happen 😔 So if you could please make an issue with a minimal example that would be fantastic 🙏🏻
'cc @Vlad Savelyev
c
Thanks for the prompt answer! Here is a capture of the console. You can reproduce the MultiQC output file by running the
test
profile of the
pairgenomealign
pipeline on the branch
multiqc_1.28
. You can run the same test with multiqc 1.27 on the
2.0.0
branch, which differse only by the module update.
p
Huh, strange. Any chance that you could send me just the HTML file? Saves me faffing around with the pipeline if you have it handy.
c
Please let me know if I need to delete the post after you downloaded the file to save space.
👀 1
p
Thanks! From Slack you mean? Should be no problem.
👍 1
ooh weird
No javascript error, you're right. Then what on earth is going on 🤔
It's like the JS rendering is stuck in an infinite loop or something
v
The h3 element and the plot wrapper have the same
id
So the plot is rendered inside the header instead of the placeholder
😂 1
Why - I can't figure out yet
Would love the original input file/config to help reproduce...
Because even if I force the section anchor and the plot anchor the same in the custom content file, the ids still should be deduplicated (see that
-1
suffix in the first line)
p
@Charles Plessy I can't find the branch
multiqc_1.28
- https://github.com/nf-core/pairgenomealign/branches/all
v
Ah, hold on. Managed to reproduce
A configuration like this does it: # id: custom_bargraph_tsv # pconfig: # anchor: 'custom_bargraph_tsv' Will fix
c
Sorry I pushed the wrong repo. The branch is there now.
p
@Vlad Savelyev bingo: 1, 2
Thanks @Charles Plessy 🙏🏻 No need now I think as @Vlad Savelyev managed to figure it out and reproduce it
We will push a fix to prevent this from happening, but really the root cause is an error in your custom content data.
Easy to fix: just change these lines
Copy code
echo "# pconfig:" >> gc_summary_mqc.tsv
    echo "#    id: 'base_content_summary_plot'" >> gc_summary_mqc.tsv
Adding the
_plot
suffix will make that ID unique and should solve your problem
c
It worked, thank you so much!
🎉 1