Skip to content

Commit

Permalink
Enabled SpreadsheetOutput in batch mode
Browse files Browse the repository at this point in the history
  • Loading branch information
@rexissimus
rexissimus committed Jun 3, 2015
1 parent fa07e3a commit 56bfd8d
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions vistrails/packages/spreadsheet/basic_widgets.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -256,8 +256,11 @@ class SpreadsheetMode(OutputMode):

@staticmethod
def can_compute():
if get_vistrails_configuration().batch:
return False
# FIXME: The spreadsheet is currently used in batch mode
# We can disable this, but what if there are modes that only
# Works with the spreadsheet and need to be written to file?
#if get_vistrails_configuration().batch:
# return False
return True

def create_display_event(self, output_module, configuration,
Expand Down

1 comment on commit 56bfd8d

@remram44
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This basically breaks batch mode.

As discussed during meeting, it would be best not to need the spreadsheet in batch mode at all -- solutions I see are:

Also identified #1089, will look into it.

Please sign in to comment.