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

Add feature MDE - metadata editor for OMERO.importer #69

Merged
merged 58 commits into from
Dec 17, 2019

Conversation

sukunis
Copy link
Member

@sukunis sukunis commented Aug 23, 2019

beta-version

sukunis and others added 24 commits August 21, 2019 09:34
Tag version 5.5.3

# gpg: Signature made Tue Jul  9 10:15:01 2019
# gpg:                using RSA key 9A8A438C637D3EF14CA10F7A43FA723866BFA9D2
# gpg: Can't check signature: No public key
# Conflicts:
#	src/main/java/org/openmicroscopy/shoola/agents/fsimporter/view/ImporterComponent.java
@pwalczysko
Copy link
Member

pwalczysko commented Aug 26, 2019

@sukunis Hello, this is great and thank you

I was wondering if you could point me to a wokflow with concrete fields filled in and a pointer as to where the intput will be located. I found https://www.ibios.uni-osnabrueck.de/index.php?cat=Service&file=OMEROImporter_v5.2_HandsOn.pdf but in the step where the metadata are added are not detailed enough. Thank you

Edit: I am only able to ever see something like in the screenshot below - whatever I try to add as metadata (defining a new model and applying it)

Screen Shot 2019-08-26 at 14 44 31

Edit2: I also have in the log of insight following lines:

-- load MDE configuration
-- init NEW MDEConfiguration
ERROR: can't find configuration file for MDE
-- init DEFAULT content [ModuleController]

So maybe I need a configuration file for MDE ? How to get one please ?

@sukunis
Copy link
Member Author

sukunis commented Aug 26, 2019

@pwalczysko
I have add more detailed description to the Readme of my branch. An example configuration file is stored in the repository. You have to put it under /omero/ . If no configuration file is available, default OME objects are loaded. If you add some annotations in the MDE, after import they should be available under key-value-pairs.

@sukunis
Copy link
Member Author

sukunis commented Aug 26, 2019

sorry: the path is /<user >/omero ( where the logfiles directory is located)

@pwalczysko
Copy link
Member

Thank you @sukunis

So I have the .xml file now and understood how to create Key-Value pairs using your app.

Screen Shot 2019-08-27 at 15 58 44

I also thought that I can apply the settings on a whole folder in one go. Is that true ? At the moment, I am getting no Key-Value pairs if I select a whole subfolder in local hierarchy in your app and apply the new settings to it (see screenshot below, I do have success though if I select just one single image, see screenshot above)

Screen Shot 2019-08-27 at 16 09 30

Is there a possibility to add the metadata to several images in one click ?

@snoopycrimecop
Copy link
Member

Conflicting PR. Removed from build OMERO-insight-push#85. See the console output for more details.
Possible conflicts:

  • Upstream changes
    • src/main/java/org/openmicroscopy/shoola/env/LookupNames.java

--conflicts

@snoopycrimecop
Copy link
Member

snoopycrimecop commented Oct 8, 2019

Conflicting PR. Removed from build OMERO-insight-push#97. See the console output for more details.
Possible conflicts:

  • Upstream changes
    • src/main/java/org/openmicroscopy/shoola/env/LookupNames.java

--conflicts Conflict resolved in build OMERO-insight-push#99. See the console output for more details.

@jburel
Copy link
Member

jburel commented Oct 21, 2019

@sukunis I have replaced the MonitorAndDebug class by ConsoleLoggerImpl class. This can also be configured in container.xml or containerImporter.xml by setting <entry name="/services/LOG/output/console" type="boolean">false</entry> to true

@jburel
Copy link
Member

jburel commented Dec 17, 2019

Thanks @sukunis. Merging so it is included in the coming 5.6 version
so you won't have to maintain a separate branch

@jburel jburel merged commit 0e1d81d into ome:master Dec 17, 2019
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.

4 participants