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

Convert to ome.xml.meta.* #18

Open
hinerm opened this issue Mar 26, 2014 · 3 comments
Open

Convert to ome.xml.meta.* #18

hinerm opened this issue Mar 26, 2014 · 3 comments

Comments

@hinerm
Copy link
Member

hinerm commented Mar 26, 2014

Currently we are still using loci.formats.meta.MetadataStore, etc... classes.

This is more restrictive than ome.xml.meta (as the loci.formats.meta equivalents are dummy subclasses).

However, let's wait to do so until we have an OMEXMLService that operates on ome.xml.meta classes, to avoid casting headaches.

@ctrueden
Copy link
Member

@melissalinkert: Our goal is for the scifio-ome-xml component to be able to depend only on ome:ome-xml and not ome:formats-api. The 5.0.0 release got things much closer to that (thanks!), but there are still a couple of little things:

  • Move loci.formats.meta.MetadataConverter to ome.xml.meta.
  • Move loci.formats.MetadataTools to ome.xml.meta.
  • Move loci.formats.services.OMEXMLService functionality to ome-xml component.

And maybe others; I didn't check 100% thoroughly yet. Basically, anything that does not need loci.formats.* should live in ome-xml IMO. Do you agree?

Anyway, this issue is a subset of that goal.

@melissalinkert
Copy link

That list should be covered by: http://trac.openmicroscopy.org.uk/ome/ticket/11981

If there is anything else that you think should be moved, could you please add it to that ticket?

@ctrueden
Copy link
Member

ctrueden commented Apr 7, 2014

Thanks Melissa, I added a comment to that ticket enumerating the list of loci.* classes upon which scifio-ome-xml currently depends.

@hinerm hinerm assigned hinerm and unassigned hinerm Mar 23, 2015
@hinerm hinerm added this to the unscheduled milestone Mar 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants