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

Addapt script to work with the new File Annotation scripts utility. #13

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

Conversation

emilroz
Copy link
Member

@emilroz emilroz commented Jul 17, 2015

No description provided.

@@ -85,15 +61,16 @@ def populate_metadata(client, conn, script_params):
description="Choose source of images",
values=dataTypes, default="Plate"),

scripts.String(
scripts.Long(

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be a List instead of Long?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The current implementation works only for a single object: https://github.com/emilroz/omero-user-scripts/blob/populate_metadata_fa_additions/util_scripts/Populate_Metadata.py#L35

That's mainly because the metadata is usually associated with a single screen or plate. I can change it to work for multiple objects if needed.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, should be fine, was just wondering.

chris-allan pushed a commit that referenced this pull request Jun 5, 2020
Update docs URL in README
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.

2 participants