Skip to content

Conversation

emilroz
Copy link
Member

@emilroz emilroz commented Jul 17, 2015

No description provided.

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