InCell 1000/2000 field count and plane metadata fixes #3898
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #3895 and https://trello.com/c/gFmUKSp6/362-incell-exposure-times
showinf -nopix -omexml
on the .xdce file in QA 31270 should show an exception similar to what is noted in #3895. With this PR, the same command should show 8 series, each of which is a field in well A1.That's almost but not quite correct with respect to what's in the .xdce. The .xdce defines 8 fields, but each field is only present in one of 3 wells (A1, A2, A3). So 8 total series makes sense, but the
Well
/WellSample
mapping does not. We would need to decide how to best represent sparse fields here - does 24 series (16 of which are blank) make sense instead? Or is it better to have 8 series and ignore the fact that they are 8 distinct fields?QA 31270 also exposes the exposure time issue noted in https://trello.com/c/gFmUKSp6/362-incell-exposure-times, so the last 2 commits here address that problem as well. This requires some configuration updates, which are in a forthcoming PR.
Opening as draft for now since this requires some discussion around sparse field handling.