Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tiff Unpacker Utility Script #1998

Merged
merged 1 commit into from
Jan 2, 2024
Merged

Conversation

JackEAllen
Copy link
Collaborator

Issue

Currently it is not possible to unpack a tiff container directly into MI I have create the issue #1990 which will be later absorbed into an EPIC to refactor how MI loads data.

IMAT are using a new camera which produces TIFF containers therefore there is a need to be able to make these compatible with mantid imaging.

As a temporary work around, I have created a quick utility script to allow users to unpack tiffs into a directory.

Testing

Manually tested the script using argument flags in different combinations.

Acceptance Criteria

  • Script can be used to unpack a tiff container into a new directory.
  • A user can select a range in which to copy images within the stack.
  • A user can set the naming convention for the images unpacked.
  • unpacked tiff container can be loaded into MI and viewed within the Spectrum Viewer.

Documentation

NA as this is a temporary script which should be removed once MI's loading has been refactored to handle tiff containers.

@JackEAllen JackEAllen marked this pull request as ready for review December 14, 2023 16:24
@coveralls
Copy link

coveralls commented Dec 14, 2023

Coverage Status

coverage: 75.698%. remained the same
when pulling 8f268fc on 1990-tiff_container_util
into bd2510a on main.

@JackEAllen JackEAllen force-pushed the 1990-tiff_container_util branch from 4dccb66 to 7112f46 Compare January 2, 2024 08:57
@JackEAllen JackEAllen self-assigned this Jan 2, 2024
@JackEAllen JackEAllen force-pushed the 1990-tiff_container_util branch from 7112f46 to e22e726 Compare January 2, 2024 11:48
@JackEAllen JackEAllen force-pushed the 1990-tiff_container_util branch from e22e726 to 8f268fc Compare January 2, 2024 11:59
Copy link
Collaborator

@samtygier-stfc samtygier-stfc left a comment

Choose a reason for hiding this comment

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

Thanks

@samtygier-stfc samtygier-stfc added this pull request to the merge queue Jan 2, 2024
Merged via the queue into main with commit 73c613a Jan 2, 2024
8 checks passed
@samtygier-stfc samtygier-stfc deleted the 1990-tiff_container_util branch January 2, 2024 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants