Ticket #1226 (closed bug: fixed)

Opened 2 years ago

Last modified 21 months ago

'Reporting' no longer works in Bioinformatics add-on

Reported by: ales Owned by: ales
Milestone: Component: library
Severity: major Keywords:
Cc: mitar Blocking:
Blocked By:

Description

The add-on widgets use from Orange.OrangeWidgets.OWWidget ... imports which results in duplicated modules in sys.modules: OWWidget, OWReport, and Orange.OrangeWidgets.OWWidget, Orange.OrangeWidgets.OWReport, while the Orange canvas application only initializes the OWReport module.

This could be fixed by removing the use of OWReport.report global variable and instead tie the report instance to the canvas application instance. Or even better use consistent imports and avoid the problem completely.

Change History

comment:1 Changed 2 years ago by Ales Erjavec <ales.erjavec@…>

In [7ea3b3f2e9b2bca75b10fbf42204c32bc96e3eef/orange]:

Replaced 'OWReport.report' references with 'OWReport.get_instance()'.

This returns 'qApp.canvasDlg.reportWindow' if present (see #1226).

comment:2 Changed 21 months ago by ales

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.