New indexing

[tkkuehn]

Proposed changes

This indexes the output metadata based on the input metadata files, not the values in the images as it did before. Side benefit: We get a nicer error messages if there are undocumented values in the input label maps.

Types of changes

What types of changes does your code introduce? Put an x in the boxes that apply

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionalitiy)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Other (if none of the other choices apply)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!

• Changes have been tested to ensure that fix is effective or that a feature works.
• Changes pass the unit tests
• Code has been run through the poe quality task
• I have included necessary documentation or comments (as necessary)
• Any dependent changes have been merged and published

Notes

All PRs will undergo the unit testing before being reviewed. You may be requested to explain or make additional changes before the PR is accepted.

[kaitj]

Lgtm.

I wonder if it would also be beneficial to include a warning if labels are missing from the label image that exist in the metadata. Can discuss at next meeting.

[tkkuehn]

I wonder if it would also be beneficial to include a warning if labels are missing from the label image that exist in the metadata.

I considered this but opted against it for now. Added to the agenda for next meeting.