Skip to content

LIMS-2208: Show dose on UDC strategy results#1076

Merged
ndg63276 merged 2 commits into
pre-release/2026-R3.2from
improvement/LIMS-2208/show-dose-on-udc-strategy
Jun 16, 2026
Merged

LIMS-2208: Show dose on UDC strategy results#1076
ndg63276 merged 2 commits into
pre-release/2026-R3.2from
improvement/LIMS-2208/show-dose-on-udc-strategy

Conversation

@ndg63276

@ndg63276 ndg63276 commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

JIRA ticket: LIMS-2208

Summary:

Add some more details to the UDC Strategy results

Changes:

  • Display subwedge dose if non-zero, and hide exposure time if zero
  • Show the total rotation as well as per image rotation
  • Hide the 'relative transmission' column, only show 'absolute transmission', rename as '(Max) transmission'.
  • Add some highlighting to rows to show which wedges belong together

To test:

  • Go to a data collection with a result from the UDC strategy, eg /dc/visit/nt44198-3/id/23192087
  • Open the Strategies bar to display the UDC Strategy results
  • Check total rotation, max transmission and dose are displayed
  • Check relative transmission and exposure time are hidden
  • Check the 'Native' wedges are coloured slightly darker to show they belong together

var highlight = true;
var lastId = null;

_.each(STRATS, function(r, i) {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we rewrite this with map or a native function? Just so we don't add any more things using underscore and make updating it harder

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, good idea.

@ndg63276 ndg63276 changed the base branch from master to pre-release/2026-R3.2 June 16, 2026 09:16
@ndg63276 ndg63276 merged commit eff1d3f into pre-release/2026-R3.2 Jun 16, 2026
2 checks passed
@ndg63276 ndg63276 deleted the improvement/LIMS-2208/show-dose-on-udc-strategy branch June 16, 2026 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants