Skip to content

Conversation

ashleymeigh2
Copy link
Collaborator

Part of #2626

Description

Improved logging for filter operations:

Logs include operation, shape, duration, and parameters

Stack selection changes are now logged

Preview operations are excluded from logs

@coveralls
Copy link

coveralls commented Jun 26, 2025

Coverage Status

coverage: 70.885% (+0.02%) from 70.868%
when pulling 0491df6 on 2626-logging-op-final
into 28b308b on main.

@samtygier-stfc samtygier-stfc self-assigned this Jun 26, 2025
@samtygier-stfc samtygier-stfc marked this pull request as draft June 26, 2025 16:27
@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
@github-project-automation github-project-automation bot moved this from Review to Todo 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 force-pushed the 2626-logging-op-final branch from fad3a27 to 0491df6 Compare June 27, 2025 16:57
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.

I squashed the commits

@samtygier-stfc samtygier-stfc marked this pull request as ready for review June 27, 2025 16:58
@samtygier-stfc samtygier-stfc enabled auto-merge June 27, 2025 16:58
@samtygier-stfc samtygier-stfc added this pull request to the merge queue Jun 27, 2025
Merged via the queue into main with commit 59e985f Jun 27, 2025
8 checks passed
@samtygier-stfc samtygier-stfc deleted the 2626-logging-op-final branch June 27, 2025 17:47
@github-project-automation github-project-automation bot moved this from Todo to Done in Mantid Imaging Work Jun 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release essential Essential to the release - showstoppers. If not done the release might be delayed
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants