jdidion / atropos-multiqc

MultiQC plugin for Atropos. (beta)
Other
0 stars 1 forks source link

Atropos module base class doesn't evaluate properly after merging #1

Open tanaes opened 7 years ago

tanaes commented 7 years ago

I went ahead and added your code from this independent module library to my fork of the main MultiQC repo here.

Also added a set of some example test data to a branch of MultiQC_TestData.

Running with a Python2-based MultiQC installation doesn't seem to work properly, I think because there's a new-style vs old-style class problem with super():

[ERROR  ]         multiqc : Oops! The 'atropos_post' MultiQC module broke... 
                    Please copy the following traceback and report it at https://github.com/ewels/MultiQC/issues 
                    (if possible, include a log file that triggers the error) 
============================================================
Module atropos_post raised an exception: Traceback (most recent call last):
  File "/Users/jonsanders/Development/git_sw/MultiQC/scripts/multiqc", line 412, in multiqc
    output = mod()
  File "/Users/jonsanders/Development/git_sw/MultiQC/multiqc/modules/atropos/atropos.py", line 380, in __init__
    super().__init__(phase='Post', **kwargs)
TypeError: super() takes at least 1 argument (0 given)
============================================================

I can get it to run and generate a report with Python3-based installation, but none of the plots actually show up. I'm getting the section headings (and subheadings for the Stats modules), but no content.

Any ideas?

tanaes commented 7 years ago

Updated with report screenshot:

screenshot