cctbx_project
cctbx_project copied to clipboard
Fix bug in merging_statistics.as_cif_block function
Upon iterating through the name dict, the test for finding the string 'percent' should be on the keys, not the values.
Added a line to a test to verify. This issue means that when the merging stats were converted to a cif block, the _reflns.percent_possible_obs
item has been in the range 0 -> 1 rather than 0 -> 100 as intended.
/azp run Full
/azp run Base
Azure Pipelines successfully started running 1 pipeline(s).
Azure Pipelines successfully started running 1 pipeline(s).
Looks good to me, thanks!
Can be merged once @rjgildea approves.
@rjgildea just noticed this is still open, care to take a quick look over?