DiamondLightSource / SynchWeb

ISPyB web application
http://diamondlightsource.github.io/SynchWeb/
Apache License 2.0
15 stars 31 forks source link

LIMS-1170: Dont use thumbnails for crystal snapshots #784

Closed ndg63276 closed 2 months ago

ndg63276 commented 3 months ago

Superseding https://github.com/DiamondLightSource/SynchWeb/pull/703

JIRA ticket: LIMS-1170

Summary:

Use full-sized crystal snapshot image files, not thumbnails.

Note that I have only touched the crystal snapshot image handling, not the diffraction preview image handling, which also uses a thumbnail image, which is generated upstream by a different mechanism to the snapshot image thumbnails.

Changes:

Remove ternary operation to switch between reading the full-sized image and the thumbnail. Always use the full-sized image. I have removed the option to use the /f/1 URL parameter as it is no longer needed, and thus removed it every time it is used.

To test:

Find a data collection that has multiple image snapshots (each DC can have up to four).

@benjaminhwilliams, can you check you are happy with this?