Skip to content

logging exporting images #2681

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jun 30, 2025
Merged

logging exporting images #2681

merged 3 commits into from
Jun 30, 2025

Conversation

ashleymeigh2
Copy link
Collaborator

@ashleymeigh2 ashleymeigh2 commented Jun 27, 2025 •
edited
Loading

Issue

Closes #2626

Description

Improved logging for filter operations:

  • Logs now include:
    • Operation name
    • Data shape
    • Duration
    • Parameters used
  • Stack selection changes are logged.

@coveralls
Copy link

coveralls commented Jun 27, 2025 •
edited
Loading

Coverage Status

coverage: 70.852% (-0.1%) from 70.975%
when pulling e45b2d3 on 2626-logging-export-files
into 00c1503 on main.

@ashleymeigh2 ashleymeigh2 moved this to In Progress in Mantid Imaging Work Jun 27, 2025
@ashleymeigh2 ashleymeigh2 moved this from In Progress to Review in Mantid Imaging Work Jun 27, 2025
@ashleymeigh2 ashleymeigh2 added the Release essential Essential to the release - showstoppers. If not done the release might be delayed label Jun 27, 2025
@samtygier-stfc samtygier-stfc self-assigned this Jun 27, 2025
samtygier-stfc
samtygier-stfc requested changes Jun 27, 2025
View reviewed changes
@github-project-automation github-project-automation bot moved this from Review to Todo in Mantid Imaging Work Jun 27, 2025
@ashleymeigh2 ashleymeigh2 moved this from Todo to Review in Mantid Imaging Work Jun 27, 2025
samtygier-stfc
samtygier-stfc approved these changes Jun 30, 2025
View reviewed changes
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.

Tested locally

@samtygier-stfc samtygier-stfc added this pull request to the merge queue Jun 30, 2025
Merged via the queue into main with commit 1c893a9 Jun 30, 2025
8 checks passed
@samtygier-stfc samtygier-stfc deleted the 2626-logging-export-files branch June 30, 2025 12:50
@github-project-automation github-project-automation bot moved this from Review to Done in Mantid Imaging Work Jun 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@samtygier-stfc samtygier-stfc samtygier-stfc approved these changes

Assignees

@samtygier-stfc samtygier-stfc

Labels
Release essential Essential to the release - showstoppers. If not done the release might be delayed
Projects
Mantid Imaging Work
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Mantid Imaging Logging: Specific Issues & Requirements
3 participants
@ashleymeigh2 @coveralls @samtygier-stfc