Skip to content

Conversation

@AndreaNathansen
Copy link
Contributor

@AndreaNathansen AndreaNathansen commented Nov 25, 2025

Description:

  • Previously, when exporting an annotation volume layer as Zarr, the paths of the mags and attachments in the properties contained the layer name of the downloaded intermediate volume data zip. As a result, the exported volume layer could not be displayed in WK afterwards. This is now fixed.

Todos:

Make sure to delete unnecessary points or to check all before merging:

  • Updated Changelog

@AndreaNathansen AndreaNathansen requested a review from fm3 November 25, 2025 18:08
@github-actions
Copy link

github-actions bot commented Nov 25, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
10160 8537 84% 80% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
webknossos/webknossos/annotation/volume_layer.py 97% 🟢
TOTAL 97% 🟢

updated for commit: ee40ea4 by action🐍

@AndreaNathansen AndreaNathansen self-assigned this Nov 25, 2025
Copy link
Member

@fm3 fm3 left a comment

Choose a reason for hiding this comment

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

LGTM :) I’ll open another PR for the docs problem that’s blocking the CI

Edit: that’s this one, currently in review scalableminds/webknossos#9100

@fm3
Copy link
Member

fm3 commented Nov 26, 2025

@AndreaNathansen CI is green now with the webknossos PR merged. Feel free to merge this :)

@AndreaNathansen AndreaNathansen enabled auto-merge (squash) November 26, 2025 16:00
@AndreaNathansen AndreaNathansen merged commit 0881e93 into master Nov 26, 2025
23 checks passed
@AndreaNathansen AndreaNathansen deleted the fix-export-annotation branch November 26, 2025 16:19
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.

3 participants