Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ROI CLI plugin proposal #80

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

dominikl
Copy link
Member

Proposal for a 'roi' CLI plugin, open for discussion.

@dominikl
Copy link
Member Author

dominikl commented Nov 20, 2017

As @mtbc just mentioned that on the call: ROIs could be attached to multiple images and part of multiple folders. So image id and folder id should actually be a list of Ids. Is it worth taking that into account or not relevant for 'real-world' use cases?

@mtbc
Copy link
Member

mtbc commented Nov 20, 2017

Glencoe may have some thoughts here. At present the OMERO model is such that if you import OME-XML with a ROI on two images then (if I recall correctly) it ends up being on just one of them. I don't know how we plan to align the two models. (Folders should indeed work fine though.)

@manics
Copy link
Member

manics commented Nov 20, 2017

Is there any existing work on serialising ROIs/Shapes as json that you could use instead of defining a new format?

@dominikl
Copy link
Member Author

If there is a short/nice json definition for a shape, happy to use that. Otherwise typing in a huge, complicated json string on the command line would be quite painful.

@manics
Copy link
Member

manics commented Nov 20, 2017

There have already been two formats in the past for defining points: https://trello.com/c/hJLlf1Xr/47-error-on-preview-rois can we at least make this new format something that's standard across most apps?

@manics
Copy link
Member

manics commented Nov 22, 2017

A couple more thoughts about defining points in the form 10 10 20 20 30 30:

  • it's prone to error if this is aimed at manual entry, since there's nothing to distinguish numbers inside a tuple from those outside, e.g. if a user inadvertently misses out one number there are no visual cues to help find where the error is
  • if we support proper 3D ROIs/points in future you can't distinguish between 2-tuples and 3-tuples

@dominikl
Copy link
Member Author

Shall we just forget about the manual entry and only have the import from a batch file option, which then can contain json or whatever existing ROI format we have?

@joshmoore
Copy link
Member

There's also this format -- https://github.com/CellMigStandOrg/biotracks -- which is basically "read from a CSV" which may be appropriate here.

cc: @rleigh-codelibre

@dominikl
Copy link
Member Author

So the current idea is, that the ROI plugin should extract the ROIs from the HDF file, which Roger's IDR tool will generate. Right? That means plugin will be very specific to the IDR case, nobody else would really be able to use it in a different scenario.

@rleigh-codelibre
Copy link
Contributor

For the HDF I was going to create, I didn't envision this being anything but a one-off screen-specific file for processing with one-off scripts for creating it and then feeding the data into OMERO. While it's possible it might have parts we could reuse for other data import tasks, I don't think it will be all reusable.

@joshmoore
Copy link
Member

Probably best to get the first example in place and we'll iterate from there. @rleigh-codelibre : if you don't have the code anywhere else, https://github.com/IDR/idr-metadata/tree/master/features might be a good starting point.

@dominikl
Copy link
Member Author

I guess with IDR/idr-metadata#292 this design PR can be closed now. Right?

@joshmoore
Copy link
Member

Let's discuss. Some decisions to make on how this repo is used.

@joshmoore
Copy link
Member

see #83 (comment) which could imply this is 006 :)

@rleigh-codelibre
Copy link
Contributor

See #84 for the suggested numbering and naming conventions.

@sbesson
Copy link
Member

sbesson commented Feb 9, 2018

As per #85, feel free to rename this as OME008/index.md.

@dominikl dominikl force-pushed the cli_roi_plugin_design_issue branch from a4624ce to aa8cb47 Compare February 16, 2018 16:04
@dominikl
Copy link
Member Author

Still, I think it can actually be closed. The hdf files are so project specific, don't know how there could be a generic roi plugin.

@joshmoore
Copy link
Member

Did this change to a ROIReader proposal? 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants