-
Notifications
You must be signed in to change notification settings - Fork 4
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 metadata import section #6
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not clear to me who this guide is targetted at:
- People using the outreach server during a workshop given by us?
- People using their own OMERO server?
It's not clear to me who this guide is targetted at:
Both. We need simply both. And we do not have the margin to produce two parallel lines. |
I think you need to tell people how to obtain and import For example, https://omero-guides.readthedocs.io/en/latest/introduction/docs/annotate.html says
Perhaps you could annotate those images instead? |
This corresponds to simple-annotation.csv For example, run: omero metadata populate --context bulkmap --cfg simple-annotation-bulkmap-config.yml Dataset:2066 To delete the map-annotations added by the above config file run: omero metadata populate --context deletemap --cfg simple-annotation-bulkmap-config.yml Dataset:2066
pwalczysko#1 adds a |
Add example simple-annotation-bulkmap-config.yml
Co-authored-by: Mark Carroll <[email protected]>
Co-authored-by: Mark Carroll <[email protected]>
Co-authored-by: Mark Carroll <[email protected]>
@manics thank you for the After reading the description in the commit and the comments in the file itself, I have two questions. Can we have the KVPs listed under the "Default namespace" organized in the "Others" header in the webclient ? What governs this ? @francesw was meaning that this might be the indentations of the blocks such as d9acb71#diff-e2d81d8cdbf9ac42c679d54ca6b0af06R51 (in her example these blocks had indentations, in yours d9acb71#diff-e2d81d8cdbf9ac42c679d54ca6b0af06R51 they are unindented. Is that the crucial point which decides about the "Others" header in webclient later ? We do not have all the mapr categories defined in the outreach servers, and, somehow, the mapr on these servers does not seem to work for these newly added KVPs using the This (and also the fact that mapr is complex, needs an extra install, we do not have a walkthrough for it, etc.) leads me to another question: Can I have a nice header like Phenotypes for my KVP group, but not have a mapr category defined ? (as you see, I would like to offer a step 1. no-mapr-yet and step 2. mapr-if-you-want approach). |
@manics Is this jinja magic d9acb71#diff-e2d81d8cdbf9ac42c679d54ca6b0af06R107 logic encoded in the omero metadata plugin ? This is a nice feature. |
Addressed the comments. Now we have two options, one using the IDR data, one using the siRNAi-HeLa data, which are easier to get. |
maybe create a folder with the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me
Co-authored-by: Simon Li <[email protected]>
Adding metadata import section.
Possibly the order could be reversed, and possibly a new section showing how to create a quick and easy, simple bulkmap config could come in.
cc @francesw @manics @jburel