[AGIPD] [CORRECT] `keep_dims` if only one cellId is corrected

This fixes the issue from this ticket https://redmine.xfel.eu/issues/143284

Description

This was an old MR to solve correcting 1 cellId for AGIPD: /gpfs/exfel/exp/MID/202230/p900249/usr/Reports/r198/MID_DET_AGIPD1M-1_correct_900249_r198_220308_223408.pdf

1. EXtra_data creates arrays without the cell dimension. Use keep_dims
2. Remove temporary workaround.
3. This fixes a bug faced when use_litframe_finder is on. As the cell_sel is switched from the expected CellRange to LitFrameSelection: /gpfs/exfel/exp/MID/202231/p900310/usr/Reports/r400/MID_DET_AGIPD1M-1_correct_900310_r400_221111_011350.pdf

How Has This Been Tested?

Reference runs and processing run 198 with 1 cellId as

Relevant Documents (optional)

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Checklist:

Reviewers

@schmidtp @roscar @kluyvert

changed the description

changed title from fix plots based on selected cellIds to [AGIPD] [CORRECT] Fix plots based on selected cellIds

added 1 commit

• a5cef9bf - fix plots based on selected cellIds

Compare with previous version

added 1 commit

• 3de9bf3b - fix plots based on selected cellIds

Compare with previous version

changed milestone to %3.5.1

As mentioned in the other MR, this is pending the EXtra-data PR. I'd also like to avoid going to 1.10, so we'll need an 1.9.x backport.

added Waiting for review label

mentioned in merge request !642 (merged)

added 1 deleted label

removed milestone %3.5.1

added 551 commits

• 3de9bf3b...7b538f32 - 550 commits from branch master
• 4ef1499b - fix plots based on selected cellIds

Compare with previous version

added 1 commit

• 25b040c8 - fix plots based on selected cellIds

Compare with previous version

removed 1 deleted label

added 1 commit

• d784162e - remove workaround for keeping dims

Compare with previous version

changed the description

resolved all threads

changed title from [AGIPD] [CORRECT] Fix plots based on selected cellIds to [AGIPD] [CORRECT] {+keep_dims if only one cellId is corrected+}

changed the description

Uhm, nice necro... is this related to https://redmine.xfel.eu/issues/143285?

added 1 commit

• 6771c5b8 - comment on the need for a first full trainId and remove iter

Compare with previous version

resolved all threads

Thanks, LGTM.

Thank you @schmidtp for the review

merged

mentioned in commit 4c881a11