Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #1716 (closed)

Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

StringColumn broken without size parameter

Reported by: jamoore Owned by: jamoore
Priority: blocker Milestone: OMERO-Beta4.2
Component: Services Version: 4.1
Keywords: n.a. Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2010-03-05 (4)

Description

I have this setup do you see anything wrong with this.. generates an error for me
43:51
    // Creates a number of empty rows of [rows] size for the base history table
    private Column[] createBaseColumns(int rows) {
        Column[] newColumns = new Column[3];
        newColumns[BASE_UID_COLUMN] = new LongColumn("Uid", "", new long[rows]);
        newColumns[BASE_DATETIME_COLUMN] = new StringColumn("DateTime", "", new String[rows]);
        newColumns[BASE_STATUS_COLUMN] = new StringColumn("Status", "", new String[rows]);
        return newColumns;
    }
44:00
    private void addBaseTableRow(String import_status) {
        
        try
        {
            Column[] newColumns = createBaseColumns(1);
            
            LongColumn uid = (LongColumn) newColumns[BASE_UID_COLUMN];
            uid.values[0] = lastIndex + 1;
            
            StringColumn date = (StringColumn) newColumns[BASE_DATETIME_COLUMN];
            date.values[0] = sqlDateTimeFormat.format(new Date());
            
            StringColumn status = (StringColumn) newColumns[BASE_STATUS_COLUMN];
            status.values[0] = import_status;     

            baseTable.addData(newColumns);                
        } catch (Throwable t) {
            throw new RuntimeException(t);
        }
    }
44:16
Caused by: Ice.UnknownException
    unknown = "Traceback (most recent call last):
                 File "/home/jboss/omero-4.1.1/lib/python/omero/columns.py", line 346, in create
                   return self.f()
                 File "/home/jboss/omero-4.1.1/lib/python/omero/columns.py", line 366, in <lambda>
                   StringColumnI: ObjectFactory(StringColumnI, lambda: StringColumnI()),
                 File "/home/jboss/omero-4.1.1/lib/python/omero/columns.py", line 202, in __init__
                   AbstractColumn.__init__(self)
                 File "/home/jboss/omero-4.1.1/lib/python/omero/columns.py", line 48, in __init__
                   d = self.descriptor(0)
                 File "/home/jboss/omero-4.1.1/lib/python/omero/columns.py", line 205, in descriptor
                   return tables.StringCol(pos=pos)
                 File "/usr/lib64/python2.5/site-packages/tables-2.1.2-py2.5-linux-x86_64.egg/tables/description.py", line 221, in __init__
                   atombase.__init__(self, *args, **kwargs)
               TypeError: __init__() takes at least 2 arguments (1 given)
               "
    at IceInternal.Outgoing.invoke(Outgoing.java:147)
    at omero.grid._TableDelM.addData(_TableDelM.java:88)
    at omero.grid.TablePrxHelper.addData(TablePrxHelper.java:116)
    at omero.grid.TablePrxHelper.addData(TablePrxHelper.java:88)
    at ome.formats.importer.gui.HistoryTableStore.addBaseTableRow(HistoryTableStore.java:136)
    ... 1 more
44:42
not the final version this has been stripped down a bit when I started debugging it but this is the basics
45:26
it fails at the baseTable.addData(newColumns);

Change History (7)

comment:1 Changed 14 years ago by jmoore

r5935 (trunk) has this patch. After testing, will need to be committed to the b_4_1 branch.

comment:2 Changed 14 years ago by jmoore

  • Type changed from Bug to Task

comment:3 Changed 14 years ago by jmoore

  • Milestone changed from OMERO-Beta4.x-HCSPreview to OMERO-Beta4.2
  • Sprint set to 2010-03-19 (5)

comment:4 Changed 14 years ago by jmoore

  • Remaining Time set to 0.1
  • Sprint changed from 2010-03-19 (5) to 2010-03-05 (4)
  • Status changed from new to assigned

comment:5 Changed 14 years ago by jmoore

  • Remaining Time changed from 0.1 to 0
  • Resolution set to fixed
  • Status changed from assigned to closed

(In [6328]) fix #1716 - Adding size to String column

original-svn-id: file:///home/svn/omero/trunk@5935 05709c45-44f0-0310-885b-81a1db45b4a6

comment:6 Changed 13 years ago by jmoore <jmoore@…>

(In [48243cf009b79d9f0ef3c3b2d0e75dc09a92e35c/ome.git]on branches 4_1, dev_4_1, dev_4_1_custom) fix #1716 - Adding size to String column

original-svn-id: file:///home/svn/omero/trunk@5935 05709c45-44f0-0310-885b-81a1db45b4a6

git-svn-id: file:///home/svn/omero/branches/Beta4.1@6328 05709c45-44f0-0310-885b-81a1db45b4a6

comment:7 Changed 13 years ago by jmoore <jmoore@…>

(In [48243cf009b79d9f0ef3c3b2d0e75dc09a92e35c/ome.git]on branches 4_1, 4_1_custom, dev_4_1, dev_4_1_custom, josh/4.1-web-backport) fix #1716 - Adding size to String column

original-svn-id: file:///home/svn/omero/trunk@5935 05709c45-44f0-0310-885b-81a1db45b4a6

git-svn-id: file:///home/svn/omero/branches/Beta4.1@6328 05709c45-44f0-0310-885b-81a1db45b4a6

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.65037 sec.)

We're Hiring!