Skip to content

populate_metadata: warn on missing image#4852

Merged
joshmoore merged 1 commit intoome:metadata52from
joshmoore:populate-missing-image
Sep 28, 2016
Merged

populate_metadata: warn on missing image#4852
joshmoore merged 1 commit intoome:metadata52from
joshmoore:populate-missing-image

Conversation

@joshmoore
Copy link
Copy Markdown
Member

What this PR does

When a well is missing from a plate, a warning is printed. The
same now happens when an image is missing from a dataset. Likely,
a --strict argument should be added which will force the existence
of all objects.

Testing this PR

  1. Check all 29 test/integration/metadata/test_populate.py tests continue passing.
  2. Attempt to annotate idr0023 (cc @eleanorwilliams). All present images should be annotated.

Related reading

When a well is missing from a plate, a warning is printed. The
same now happens when an image is missing from a dataset. Likely,
a `--strict` argument should be added which will force the existence
of all objects.
Copy link
Copy Markdown

@eleanorwilliams eleanorwilliams left a comment

Choose a reason for hiding this comment

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

Tested on idr0023 which has missing image files using OMERO.py-5.2.3-248-c181f59-ice35-b85.

Got the 3 warnings about missing files as expected but annotation.csv file is converted to bulk table as desired.

@eleanorwilliams
Copy link
Copy Markdown

Note: I don't know how to check the first point.

  1. Check all 29 test/integration/metadata/test_populate.py tests continue passing.

@joshmoore
Copy link
Copy Markdown
Member Author

joshmoore commented Sep 22, 2016

Tests tomorrow should now pass due to #4858

@joshmoore
Copy link
Copy Markdown
Member Author

@joshmoore joshmoore merged commit 592ef09 into ome:metadata52 Sep 28, 2016
@joshmoore joshmoore deleted the populate-missing-image branch September 28, 2016 13:13
@atarkowska
Copy link
Copy Markdown
Member

--rebased-to #5224

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants