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 #10904 (closed)

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

Import and metadata

Reported by: jburel Owned by: jamoore
Priority: blocker Milestone: 5.0.0-beta1
Component: Import Version: n.a.
Keywords: n.a. Cc: ux@…, fs@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: FS demo 4.2

Description (last modified by jburel)

The user will be now able to close the window after uploading the image.
This implies that annotating the image when the import is finished will no longer work.
We need to have the ability to pass annotation to the server.
The annotation will be linked to the object server-side when the import is finished.

We could use the ImportSettings object

Change History (11)

comment:1 Changed 11 years ago by jburel

  • Description modified (diff)

comment:2 Changed 11 years ago by jburel

  • Milestone changed from Unscheduled to OMERO-5

comment:3 Changed 11 years ago by jburel

  • Type changed from User Story to Task

comment:4 Changed 11 years ago by jburel

Using the ImportSettings object to specify the annotation does not work. The annotation is not linked to the imported image.
Same if you do ic.setCustomAnnotationList(l);

This is a major regression

Last edited 11 years ago by jburel (previous) (diff)

comment:5 Changed 11 years ago by jburel

  • Priority changed from minor to blocker

comment:6 Changed 11 years ago by jamoore

  • Owner set to jamoore
  • Sprint set to FS demo 4.2

Tested via CLI and basic usage works:

(ome1)jamoore@blue:/opt/ome2$ dist/bin/omero -s root@localhost -w ome import dist/insight.fake --annotation_ns FOO --annotation_text BAR
...
2013-05-23 10:52:36,619 6466       [l.Client-0] INFO   ormats.importer.cli.LoggingImportMonitor  - IMPORT_DONE Imported file: /opt/ome2/dist/insight.fake
Imported pixels:
302
Other imported objects:
Fileset:302
Image:302
2013-05-23 10:52:36,619 6466       [l.Client-0] INFO      ome.formats.importer.cli.ErrorHandler  - Number of errors: 0
(ome1)jamoore@blue:/opt/ome2$ psql -U omero -h localhost ome2
Password for user omero: 
psql (9.1.9)
SSL connection (cipher: DHE-RSA-AES256-SHA, bits: 256)
Type "help" for help.

ome2=> select * from annotation order by id desc limit 10;
ome2=> select * from annotation order by id desc limit 10;
ome2=> select * from imageannotationlink order by id desc limit 10;
 id | permissions | version | child | creation_id | external_id | group_id | owner_id | update_id | parent 
----+-------------+---------+-------+-------------+-------------+----------+----------+-----------+--------
  1 |        -120 |       0 |   302 |        2222 |             |        0 |        0 |      2222 |    302
(1 row)

ome2=> select * from annotation order by id desc limit 10;
ome2=> select * from annotation where ns = 'FOO' order by id desc limit 10;
    discriminator     | id  | description | permissions | ns  | version | boolvalue | textvalue | doublevalue | longvalue | termvalue | timevalue | creation_id | external_id | group_id | owner_id | update_id | file 
----------------------+-----+-------------+-------------+-----+---------+-----------+-----------+-------------+-----------+-----------+-----------+-------------+-------------+----------+----------+-----------+------
 /basic/text/comment/ | 302 |             |        -120 | FOO |         |           | BAR       |             |           |           |           |        2222 |             |        0 |        0 |      2222 |     
(1 row)

Now looking into insight.

comment:7 Changed 11 years ago by jburel

At the moment the code has been removed,
i can add it if required.

comment:8 Changed 11 years ago by jburel

I was looking at tags I don't think it will make a difference

comment:9 Changed 11 years ago by jamoore

  • Resolution set to fixed
  • Status changed from new to closed

comment:11 Changed 11 years ago by jburel <j.burel@…>

  • Remaining Time set to 0

(In [d45d47a02bf42769c3accf453f2c8e300c21d41b/ome.git] on branch develop) Set annotation list in OmeroImageServiceImpl? (Fix #10904)

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.69445 sec.)

We're Hiring!