Skip to content

LIMS-2033: Fix count of autoprocessing errors#1047

Open
ndg63276 wants to merge 1 commit intomasterfrom
fix/LIMS-2033/fix-count-of-autoprocessing-errors
Open

LIMS-2033: Fix count of autoprocessing errors#1047
ndg63276 wants to merge 1 commit intomasterfrom
fix/LIMS-2033/fix-count-of-autoprocessing-errors

Conversation

@ndg63276
Copy link
Collaborator

JIRA ticket: LIMS-2033

Summary:

The autoprocessing pipelines produce messages as either INFOS, WARNINGS or ERRORS. These are displayed by the data collection or group, but the number shown is wrong.

Changes:

  • Get all the messages first, via both sets of possible joins, then Union them, then Sum them only at the end
  • Ignore autoprocprogramid so the Union command correctly ignores duplicates
  • Return totals with the dataCollectionId or dataCollectionGroupId passed in, if using a DCG then sum the totals together
  • Display the total for the group if a group is displayed
  • Dont limit the page size to 15, some jobs give more than 15 messages

To test:

  • Go to a visit with a mix of grouped and ungrouped data collections, eg /dc/visit/cm40608-5/ty/fc
  • Look for the colourful little button on each data collection like this
image - Look at an ungrouped data collection, check the numbers on the button match those displayed if you click the button (summing the tabs together) - Look at a data collection group (coloured blue), check the numbers on the button match those displayed if you click the button (summing all the tabs). These numbers may be higher as they are for all processing jobs on all DCs in the group - Click into the group, check the numbers on the buttons in the group match those displayed if you click the buttons, and add up to those shown on the whole group. (Eg group shows 87 "infos", comprising of 2 data collections with 53 and 34 "infos" respectively, see /dc/visit/cm40608-5/dcg/17176266)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant