Dropping quast namespace broke `table_columns_visible`

[jchorl]

Description of bug

We used to have a config like:

table_columns_visible:
  QUAST:
    "# misassemblies": false

This commit removed the quast namespace: 056d21d#diff-6d91e36cef5a41720305bb1bd9f0ab612e3830d0cce62efc0933a1d982b5296dL252

We now need to specify this config as:

table_columns_visible:
  "":
    "# misassemblies": false

Is this expected/desirable?

Thanks!

File that triggers the error

No response

MultiQC Error log

No response

Before submitting

• I have read the troubleshooting documentation.
• I am using the latest release of MultiQC.
• I have included a full MultiQC log, not truncated.
• I have attached an input file (.zip if necessary) that triggers the error.

[vladsavelyev]

Hi @jchorl,

That worked before really by accident, as we don't normally pass the namespace to table.plot() config in modules unless we want to tell apart between different submodules/subtools (like in DRAGEN and CellRanger), and QUAST was an exception.

In the future refactoring, we'll make it more flexible, but the way it is supported now is exactly as you modified it:

table_columns_visible:
  "":
    "# misassemblies": false

I think I'll simplify the usage to:

table_columns_visible:
  "# misassemblies": false

[vladsavelyev]

Done (see #2191) - now you can also use a flat config like this:

table_columns_visible:
  "# misassemblies": false

The column IDs should be unique anyway :)