diff --git a/lib/galaxy/datatypes/metadata.py b/lib/galaxy/datatypes/metadata.py index ba3599d7589..77d3c987c8f 100644 --- a/lib/galaxy/datatypes/metadata.py +++ b/lib/galaxy/datatypes/metadata.py @@ -60,10 +60,10 @@ class MetadataParameter( object ): if isinstance( value, ListType ): for i, elem in enumerate( value ): if type ( elem ) == unicode: - value[i] = str( elem ) + value[i] = elem.decode( 'ascii' ) elif isinstance ( value, basestring ): if type( value ) == unicode: - value = str( value ) + value = value.decode( 'ascii' ) self.value = value self.context = context self.display = True @@ -76,7 +76,7 @@ class MetadataParameter( object ): @classmethod def marshal( cls, value ): ''' - This method should/can be overridden to convert the incomming + This method should/can be overridden to convert the incoming value to whatever type it is supposed to be. ''' return value @@ -273,7 +273,6 @@ class ColumnParameter( RangeParameter ): class ColumnTypesParameter( MetadataParameter ): def __init__( self, spec, value, context ): MetadataParameter.__init__( self, spec, value, context ) - def __str__(self): return ",".join( map( str, self.value ) ) diff --git a/lib/galaxy/datatypes/tabular.py b/lib/galaxy/datatypes/tabular.py index 0bcc09e5644..fddd92de944 100644 --- a/lib/galaxy/datatypes/tabular.py +++ b/lib/galaxy/datatypes/tabular.py @@ -19,8 +19,8 @@ class Tabular( data.Text ): """Tab delimited data""" """Add metadata elements""" - MetadataElement( name="columns", default=0, desc="Number of columns", readonly=True, visible=False ) - MetadataElement( name="column_types", default=[], desc="Column types", param=metadata.ColumnTypesParameter, readonly=True, visible=False ) + MetadataElement( name="columns", default=0, desc="Number of columns", readonly=True, visible=False, no_value=0 ) + MetadataElement( name="column_types", default=[], desc="Column types", param=metadata.ColumnTypesParameter, readonly=True, visible=False, no_value=[] ) def init_meta( self, dataset, copy_from=None ): data.Text.init_meta( self, dataset, copy_from=copy_from ) diff --git a/tools/stats/column_maker.py b/tools/stats/column_maker.py index 7cd81a1a888..f9a6ec03b6c 100644 --- a/tools/stats/column_maker.py +++ b/tools/stats/column_maker.py @@ -19,12 +19,10 @@ expr = sys.argv[3] round = sys.argv[4] try: in_columns = int( sys.argv[5] ) - # in_column_types is passed as a string that looks something like: - # "[u'str', u'int', u'int', u'str', u'int', u'str']" - in_column_types = sys.argv[6].strip( '[' ).strip( ']' ).replace( 'u', '' ).replace( "'", '' ).split( ',' ) + in_column_types = sys.argv[6].split( ',' ) except: stop_err( "Data does not appear to be tabular. This tool can only be used with tab-delimited data." ) - + # Unescape if input has been escaped mapped_str = { '__lt__': '<', diff --git a/tools/stats/filtering.py b/tools/stats/filtering.py index 2da9c370ed6..bb3d4d2c953 100644 --- a/tools/stats/filtering.py +++ b/tools/stats/filtering.py @@ -26,9 +26,7 @@ out_fname = sys.argv[2] cond_text = sys.argv[3] try: in_columns = int( sys.argv[4] ) - # in_column_types is passed as a string that looks something like: - # "[u'str', u'int', u'int', u'str', u'int', u'str']" - in_column_types = sys.argv[5].strip( '[' ).strip( ']' ).replace( 'u', '' ).replace( "'", '' ).split( ',' ) + in_column_types = sys.argv[5].split( ',' ) except: stop_err( "Data does not appear to be tabular. This tool can only be used with tab-delimited data." )