Skip to content

Conversation

@ndg63276
Copy link
Collaborator

@ndg63276 ndg63276 commented Nov 1, 2024

JIRA ticket: LIMS-1498

Summary:

A user emailed pointing out that the Archive download of an autoprocessing run gives a pretty abstract filename of <datacollectionid>_<pipeline>.zip, eg 106290598_xia2dials.zip

Changes:

  • Added the image prefix and data collection number, so we now have <datacollectionid>_<imageprefix>_<datacolletionnumber>_<pipeline>.zip, eg 106290598_sethau_8_42_xia2dials.zip

To test:

  • Go to a data collection with successful autoprocessing eg /dc/visit/cm37235-4/id/15531187
  • Go to a processing pipeline and click on the "Archive" button, check the download has the image prefix and data collection number in it.

@ndg63276 ndg63276 changed the base branch from master to pre-release/2024-R5.2 November 12, 2024 10:03
@ndg63276 ndg63276 merged commit 6ba748f into pre-release/2024-R5.2 Nov 12, 2024
@ndg63276 ndg63276 deleted the improvement/LIMS-1498/change-name-of-processing-archive branch November 12, 2024 10:06
ndg63276 added a commit that referenced this pull request Dec 16, 2024
…882)

* LIMS-1498: Change name of processed data archive (#855)

* LIMS-1469: Fix download button on old summary page (#840)

* LIMS-1463: Always display Mesh3D data collections as grid scans (#845)

* LIMS-1515: Add energy value to data collections (#853)

* LIMS-261: Allow download of PDB files (#857)

* LIMS-1529: Fix fast ep model viewer (#858)

* LIMS-753: Allow LN2 topups as part of dewar history (#856)

* LIMS-1530: Add To Queue Fails when Not Completed filter applied (#864)

* LIMS-1530: Add To Queue Fails when Not Completed filter applied

* LIMS-1552: Remove old jquery (#871)

* LIMS-1500: Set the dispatch request email recipients based on the dewar barcode (#848)

* LIMS-1458: Fix GitHub warnings (#841)

---------

Co-authored-by: Mark Williams <mark.williams@diamond.ac.uk>
Co-authored-by: Guilherme Francisco <guilherme.de-freitas@diamond.ac.uk>
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.

3 participants