Skip to content

Conversation

@jluethi
Copy link
Collaborator

@jluethi jluethi commented Jun 3, 2025

Closes #958

Checklist before merging

  • I added an appropriate entry to CHANGELOG.md

@jluethi
Copy link
Collaborator Author

jluethi commented Jun 3, 2025

For historical reasons, the heuristic to find the reference cycle didn't include the case where the reference image needs to find itself (we had special case handling of that case in fractal-tasks-core < 1.0.0 and the default fallback worked for most users, which is why the bug wasn't more evident before).

Now, we allow the heuristic to find itself as the reference image when the reference cycle is processed. Details see #958

@github-actions
Copy link

github-actions bot commented Jun 3, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  fractal_tasks_core/tasks
  _zarr_utils.py
Project Total  

This report was generated by python-coverage-comment-action

@jluethi jluethi merged commit 562d267 into main Jun 3, 2025
15 checks passed
@jluethi jluethi deleted the 958_apply_registration_fix branch June 3, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Registration task not finding reference image in some cases

2 participants