James Beilsten-Edmands

@rjgildea just noticed this is still open, care to take a quick look over?

@phyy-nx for the bbox z coordinates on ImageSequences, I looked at two examples. A .cbf file from I24 beamline had bbox z coordinates of -1, 0, whereas a .h5 file...

Thanks for the suggestions @biochem-fan I think it should be possible to allow both of those options, and I think that it's clunky/annoying to have to specify `exclude_images=` multiple times....

The intention here was that the numbers on the plot correspond to what you would need to input for the `exclude_images` command, so this is a bug. Thanks for reporting,...

Thanks for the link David, I think the Gemmi implementation notes summarise the issue well: "The Hall notation (-I 4bd 2c 3), first proposed by Sydney R. Hall in 1981,...

I think what you suspect is correct. We leave the cell in its input setting (doing the Patterson symmetry in the 'standard' cell on a copy of the data), with...

Thanks for the report, I wasn't aware of this. I agree it would be nice to always have the same number of columns, I'll investigate what changes in dials/cctbx would...

Thanks for reporting this. I'll take some time to look into this further, but with regards to the first comment, in the default mode, dials.scale will use profile estimates up...

I'm generally on board with this change, although I think we might need to update/expand the output as I believe some confusion could be caused (I myself was confused, thought...

Can't remember why I didn't mark this as ready at the time, I can't see anything particular missing so i'm now putting this out for review.