Prepare Release 25.09.0 #8874
Merged
Prepare Release 25.09.0 #8874
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
fm3
commented
Aug 26, 2025
fm3
commented
- updated changelog
📝 WalkthroughWalkthroughAdds a 25.09.0 release entry and migrations, and removes many unreleased changelog notes now consolidated into the release; also updates the default local docker-compose webknossos image tag. No source code or public API declarations are modified in this diff. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 9
🧹 Nitpick comments (7)
MIGRATIONS.released.md (1)
12-15: Sort Postgres Evolutions and Add Header Colon for ConsistencyAll referenced evolution files were verified to exist under
conf/evolutions/, so the proposed change can be applied safely.Affected locations:
MIGRATIONS.released.md, lines 12–15Proposed diff:
-### Postgres Evolutions -- [138-add-webauthn-credentials.sql](conf/evolutions/138-add-webauthn-credentials.sql) -- [136-extra-column-for-email-changed.sql](conf/evolutions/136-extra-column-for-email-changed.sql) -- [137-virtual-datasets.sql](conf/evolutions/137-virtual-datasets.sql) +### Postgres Evolutions: +- [136-extra-column-for-email-changed.sql](conf/evolutions/136-extra-column-for-email-changed.sql) +- [137-virtual-datasets.sql](conf/evolutions/137-virtual-datasets.sql) +- [138-add-webauthn-credentials.sql](conf/evolutions/138-add-webauthn-credentials.sql)CHANGELOG.released.md (6)
38-38: Wording: avoid “Changed that …” construction.Prefer a direct, active phrasing.
- Changed that the d/f switching setting no longer affects movements other than moving via the d/f keyboard shortcuts. [#8786](https://github.com/scalableminds/webknossos/pull/8786) + The d/f switching setting no longer affects movements other than moving via the d/f keyboard shortcuts. [#8786](https://github.com/scalableminds/webknossos/pull/8786)
40-40: Punctuation consistency (period).Most bullets end with a period. Add one here for consistency.
- Integrated "Change Email" functionality into Account Settings [#8840](https://github.com/scalableminds/webknossos/pull/8840) + Integrated "Change Email" functionality into Account Settings. [#8840](https://github.com/scalableminds/webknossos/pull/8840)
42-42: Clarify status code.Specify HTTP status code explicitly.
- Zarr streaming a volume annotation will now respond with errors (500) if the underlying data encountered errors. [#8853](https://github.com/scalableminds/webknossos/pull/8853) + Zarr streaming a volume annotation will now respond with HTTP 500 if the underlying data encountered errors. [#8853](https://github.com/scalableminds/webknossos/pull/8853)
44-44: Numeric readability.Optional: add thousands separators.
- Added protection against server outages by limiting the number of update actions in an annotation update transaction to 1000000. [#8866](https://github.com/scalableminds/webknossos/pull/8866) + Added protection against server outages by limiting the number of update actions in an annotation update transaction to 1,000,000. [#8866](https://github.com/scalableminds/webknossos/pull/8866)
57-57: Clarity: add “when”.- Fixed viewing a dataset zoomed far out which has rotations configured in the dataset settings Data Source tab. [#8819](https://github.com/scalableminds/webknossos/pull/8819) + Fixed viewing a dataset when zoomed far out with rotations configured in the dataset settings (Data Source tab). [#8819](https://github.com/scalableminds/webknossos/pull/8819)
24-24: Hyphenation consistency (“reupload”).Elsewhere in the changelog “reupload” is used without a hyphen; align here.
- Note that re-upload is not yet implemented. [#8816](https://github.com/scalableminds/webknossos/pull/8816) + Note that reupload is not yet implemented. [#8816](https://github.com/scalableminds/webknossos/pull/8816)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (42)
CHANGELOG.released.md(1 hunks)MIGRATIONS.released.md(2 hunks)unreleased_changes/8393.md(0 hunks)unreleased_changes/8614.md(0 hunks)unreleased_changes/8671.md(0 hunks)unreleased_changes/8678.md(0 hunks)unreleased_changes/8689.md(0 hunks)unreleased_changes/8708.md(0 hunks)unreleased_changes/8719.md(0 hunks)unreleased_changes/8732.md(0 hunks)unreleased_changes/8739.md(0 hunks)unreleased_changes/8757.md(0 hunks)unreleased_changes/8776.md(0 hunks)unreleased_changes/8786.md(0 hunks)unreleased_changes/8791.md(0 hunks)unreleased_changes/8793.md(0 hunks)unreleased_changes/8795.md(0 hunks)unreleased_changes/8798.md(0 hunks)unreleased_changes/8800.md(0 hunks)unreleased_changes/8801.md(0 hunks)unreleased_changes/8806.md(0 hunks)unreleased_changes/8807.md(0 hunks)unreleased_changes/8809.md(0 hunks)unreleased_changes/8816.md(0 hunks)unreleased_changes/8819.md(0 hunks)unreleased_changes/8828.md(0 hunks)unreleased_changes/8830.md(0 hunks)unreleased_changes/8832.md(0 hunks)unreleased_changes/8834.md(0 hunks)unreleased_changes/8837.md(0 hunks)unreleased_changes/8840.md(0 hunks)unreleased_changes/8847.md(0 hunks)unreleased_changes/8849.md(0 hunks)unreleased_changes/8853.md(0 hunks)unreleased_changes/8854.md(0 hunks)unreleased_changes/8861.md(0 hunks)unreleased_changes/8862.md(0 hunks)unreleased_changes/8863.md(0 hunks)unreleased_changes/8866.md(0 hunks)unreleased_changes/8867.md(0 hunks)unreleased_changes/8868.md(0 hunks)unreleased_changes/8872.md(0 hunks)
💤 Files with no reviewable changes (40)
- unreleased_changes/8868.md
- unreleased_changes/8806.md
- unreleased_changes/8866.md
- unreleased_changes/8809.md
- unreleased_changes/8819.md
- unreleased_changes/8678.md
- unreleased_changes/8798.md
- unreleased_changes/8807.md
- unreleased_changes/8786.md
- unreleased_changes/8689.md
- unreleased_changes/8840.md
- unreleased_changes/8793.md
- unreleased_changes/8837.md
- unreleased_changes/8830.md
- unreleased_changes/8862.md
- unreleased_changes/8791.md
- unreleased_changes/8853.md
- unreleased_changes/8872.md
- unreleased_changes/8795.md
- unreleased_changes/8832.md
- unreleased_changes/8834.md
- unreleased_changes/8854.md
- unreleased_changes/8719.md
- unreleased_changes/8849.md
- unreleased_changes/8863.md
- unreleased_changes/8671.md
- unreleased_changes/8614.md
- unreleased_changes/8861.md
- unreleased_changes/8739.md
- unreleased_changes/8393.md
- unreleased_changes/8828.md
- unreleased_changes/8708.md
- unreleased_changes/8847.md
- unreleased_changes/8801.md
- unreleased_changes/8732.md
- unreleased_changes/8800.md
- unreleased_changes/8776.md
- unreleased_changes/8816.md
- unreleased_changes/8757.md
- unreleased_changes/8867.md
🧰 Additional context used
🪛 LanguageTool
MIGRATIONS.released.md
[grammar] ~9-~9: There might be a mistake here.
Context: ...ossos/releases/tag/25.09.0) - 2025-08-26 [Commits](https://github.com/scalablemind...
(QB_NEW_EN)
[grammar] ~12-~12: There might be a mistake here.
Context: ...07.1...25.09.0) ### Postgres Evolutions - [138-add-webauthn-credentials.sql](conf/e...
(QB_NEW_EN)
[grammar] ~13-~13: There might be a mistake here.
Context: ...ions - 138-add-webauthn-credentials.sql - [136-extra-column-for-email-changed.sql](...
(QB_NEW_EN)
[grammar] ~14-~14: There might be a mistake here.
Context: ... 136-extra-column-for-email-changed.sql - [137-virtual-datasets.sql](conf/evolution...
(QB_NEW_EN)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: frontend-tests
- GitHub Check: build-smoketest-push
- GitHub Check: backend-tests
🔇 Additional comments (2)
MIGRATIONS.released.md (2)
894-894: LGTM on list entry normalizationThe link formatting for 012-add-foreign-keys.sql is consistent with the rest of the document. No action needed.
9-11: Confirm 25.09.0 tag and compare URLs resolve post-taggingVerification shows that both endpoints currently return HTTP 404 (the
25.09.0release tag hasn’t been pushed yet). Once you’ve created and pushed the25.09.0tag, please re-run these checks and ensure you get HTTP 200 for both:• https://github.com/scalableminds/webknossos/releases/tag/25.09.0
• 25.07.1...25.09.0You can verify with:
curl -s -o /dev/null -w 'HTTP %{http_code}\n' <url>
Comment on lines
+14
to
+16
| - Added the possibility to rotate the planes in ortho view. While rotated, volume annotation is disabled. [#8614](https://github.com/scalableminds/webknossos/pull/8614) | ||
| - Implement Authentication with Passkeys. [#8393](https://github.com/scalableminds/webknossos/pull/8393) | ||
| - Added support for training and running AI instance segmentation models from WK UI. [#8849](https://github.com/scalableminds/webknossos/pull/8849) |
There was a problem hiding this comment.
Tense/voice consistency in Highlights and Added (use past tense).
Use “Implemented …” (past tense) instead of “Implement …” to match the surrounding style. Also prefer sentence case.
- Implement Authentication with Passkeys. [#8393](https://github.com/scalableminds/webknossos/pull/8393)
+ Implemented authentication with passkeys. [#8393](https://github.com/scalableminds/webknossos/pull/8393)Apply the same change in the “Added” section:
- Implement Authentication with Passkeys. [#8393](https://github.com/scalableminds/webknossos/pull/8393)
+ Implemented authentication with passkeys. [#8393](https://github.com/scalableminds/webknossos/pull/8393)Also applies to: 19-19
🤖 Prompt for AI Agents
In CHANGELOG.released.md around lines 14-16 (and also line 19), the entries use
inconsistent tense/voice and capitalization; change "Implement Authentication
with Passkeys." to "Implemented authentication with passkeys." and adjust any
other entries to past tense and sentence case (e.g., "Added the possibility to
rotate the planes in ortho view..." is fine, ensure similar sentence-case
formatting for line 19), preserving PR links and punctuation.
| - Added the config option `features.registerToDefaultOrgaEnabled` (default true) to allow admins to disable the normal registration flow to the default organization. If it is false, SSO or invites are required. [#8807](https://github.com/scalableminds/webknossos/pull/8807) | ||
| - Editable mapping (aka supervoxel proofreading) annotations can now be downloaded and the edited edges are included in a zarr3-based format. Note that re-upload is not yet implemented. [#8816](https://github.com/scalableminds/webknossos/pull/8816) | ||
| - Added support for training and running AI instance segmentation models from WK UI. [#8849](https://github.com/scalableminds/webknossos/pull/8849) | ||
| - Added support for the layersToLink format as the webknossos-libs currently send it (datasetId as datasetName) [#8861](https://github.com/scalableminds/webknossos/pull/8861) |
There was a problem hiding this comment.
Grammar and formatting: subject–verb agreement and inline code formatting.
- “webknossos-libs currently send it” → “webknossos-libs currently sends it”.
- Consider code-formatting for identifiers.
- Added support for the layersToLink format as the webknossos-libs currently send it (datasetId as datasetName) [#8861](https://github.com/scalableminds/webknossos/pull/8861)
+ Added support for the `layersToLink` format as `webknossos-libs` currently sends it (use `datasetId` as `datasetName`). [#8861](https://github.com/scalableminds/webknossos/pull/8861)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - Added support for the layersToLink format as the webknossos-libs currently send it (datasetId as datasetName) [#8861](https://github.com/scalableminds/webknossos/pull/8861) | |
| - Added support for the `layersToLink` format as `webknossos-libs` currently sends it (use `datasetId` as `datasetName`). [#8861](https://github.com/scalableminds/webknossos/pull/8861) |
🤖 Prompt for AI Agents
In CHANGELOG.released.md around line 26, fix the grammar and identifier
formatting in the changelog entry: change “webknossos-libs currently send it” to
“webknossos-libs currently sends it” and wrap identifiers like layersToLink,
datasetId and datasetName in inline code formatting (e.g., `layersToLink`,
`datasetId`, `datasetName`) so the sentence reads cleanly and uses proper code
formatting.
| - Added support for the layersToLink format as the webknossos-libs currently send it (datasetId as datasetName) [#8861](https://github.com/scalableminds/webknossos/pull/8861) | ||
| - Added the config option `silhouette.cookieAuthenticator.sameSite` to control the relevant cookie header in order to allow iframe usage if desired for a particular WEBKNOSSOS instance. [#8862](https://github.com/scalableminds/webknossos/pull/8862) | ||
| - Updated backend dependencies for improved stability and performance. [#8863](https://github.com/scalableminds/webknossos/pull/8863) | ||
| - Add tracingstore route to export agglomerate graph as protobuf, to be used by the python client. [#8872](https://github.com/scalableminds/webknossos/pull/8872) |
There was a problem hiding this comment.
Consistency (past tense) and capitalization.
Stay consistent with past-tense bullets and capitalize “Python”.
- Add tracingstore route to export agglomerate graph as protobuf, to be used by the python client. [#8872](https://github.com/scalableminds/webknossos/pull/8872)
+ Added tracingstore route to export agglomerate graph as protobuf, to be used by the Python client. [#8872](https://github.com/scalableminds/webknossos/pull/8872)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - Add tracingstore route to export agglomerate graph as protobuf, to be used by the python client. [#8872](https://github.com/scalableminds/webknossos/pull/8872) | |
| - Added tracingstore route to export agglomerate graph as protobuf, to be used by the Python client. [#8872](https://github.com/scalableminds/webknossos/pull/8872) |
🤖 Prompt for AI Agents
In CHANGELOG.released.md around line 29, the bullet uses present tense and
lowercases "python"; update the sentence to past tense and capitalize "Python"
(e.g., change "Add tracingstore route ... to be used by the python client." to
"Added tracingstore route ... to be used by the Python client.") so the
changelog remains consistent.
| - Integrated "Change Email" functionality into Account Settings [#8840](https://github.com/scalableminds/webknossos/pull/8840) | ||
| - When adding a new dataset from UI modal, the organization's default team is now automatically pre-selected for dataset access permissions. [#8847](https://github.com/scalableminds/webknossos/pull/8847) | ||
| - Zarr streaming a volume annotation will now respond with errors (500) if the underlying data encountered errors. [#8853](https://github.com/scalableminds/webknossos/pull/8853) | ||
| - datasource-properties.jsons with wkwResolutions now do not need the cubeLength parameter anymore. [#8854](https://github.com/scalableminds/webknossos/pull/8854) |
There was a problem hiding this comment.
Pluralization and file naming.
Use “files” rather than “jsons”; keep the filename singular in code formatting.
- datasource-properties.jsons with wkwResolutions now do not need the cubeLength parameter anymore. [#8854](https://github.com/scalableminds/webknossos/pull/8854)
+ `datasource-properties.json` files with `wkwResolutions` no longer need the `cubeLength` parameter. [#8854](https://github.com/scalableminds/webknossos/pull/8854)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - datasource-properties.jsons with wkwResolutions now do not need the cubeLength parameter anymore. [#8854](https://github.com/scalableminds/webknossos/pull/8854) | |
| - `datasource-properties.json` files with `wkwResolutions` no longer need the `cubeLength` parameter. [#8854](https://github.com/scalableminds/webknossos/pull/8854) |
🤖 Prompt for AI Agents
In CHANGELOG.released.md around line 43, replace the incorrect pluralized
filename "datasource-properties.jsons" with the phrasing
datasource-properties.json files — i.e., use "files" for plural but keep the
filename itself singular (datasource-properties.json) in the text.
| - Zarr streaming a volume annotation will now respond with errors (500) if the underlying data encountered errors. [#8853](https://github.com/scalableminds/webknossos/pull/8853) | ||
| - datasource-properties.jsons with wkwResolutions now do not need the cubeLength parameter anymore. [#8854](https://github.com/scalableminds/webknossos/pull/8854) | ||
| - Added protection against server outages by limiting the number of update actions in an annotation update transaction to 1000000. [#8866](https://github.com/scalableminds/webknossos/pull/8866) | ||
| - Remove left over occurrence of `SELECT *` statement in backend. Added assertions to CI to ensure that the `SELECT *` statement is not used in the backend. [#8867](https://github.com/scalableminds/webknossos/pull/8867) |
There was a problem hiding this comment.
Tense and hyphenation.
- “Remove” → “Removed”
- “left over” → “leftover”
- Minor duplication; consider merging sentences.
- Remove left over occurrence of `SELECT *` statement in backend. Added assertions to CI to ensure that the `SELECT *` statement is not used in the backend. [#8867](https://github.com/scalableminds/webknossos/pull/8867)
+ Removed leftover occurrence of `SELECT *` in the backend and added CI assertions to ensure `SELECT *` is not used. [#8867](https://github.com/scalableminds/webknossos/pull/8867)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - Remove left over occurrence of `SELECT *` statement in backend. Added assertions to CI to ensure that the `SELECT *` statement is not used in the backend. [#8867](https://github.com/scalableminds/webknossos/pull/8867) | |
| - Removed leftover occurrence of `SELECT *` in the backend and added CI assertions to ensure `SELECT *` is not used. [#8867](https://github.com/scalableminds/webknossos/pull/8867) |
🤖 Prompt for AI Agents
In CHANGELOG.released.md around line 45, update the sentence to past tense,
combine duplicated ideas and use the single-word "leftover"; replace the current
line with a merged, corrected sentence such as: "Removed leftover occurrence of
the `SELECT *` statement in the backend and added CI assertions to prevent use
of `SELECT *` in the backend.
[#8867](https://github.com/scalableminds/webknossos/pull/8867)".
| - Fixed a bug where reading large zarr3 layer attachments would sometimes fail for some values due to an integer overflow. [#8798](https://github.com/scalableminds/webknossos/pull/8798) | ||
| - Fixed a bug where the zarr.json included in volume annotation zarr downloads would miss the compression codec even though the chunks were compressed. [#8800](https://github.com/scalableminds/webknossos/pull/8800) | ||
| - When renaming skeletons or skeleton groups, it is now possible to select a part of the name. [#8801](https://github.com/scalableminds/webknossos/pull/8801) | ||
| - The view to change ones email address is now available in the top right dropdown menu of the navbar. [#8806](https://github.com/scalableminds/webknossos/pull/8806) |
There was a problem hiding this comment.
Grammar: possessive “one’s”.
- The view to change ones email address is now available in the top right dropdown menu of the navbar. [#8806](https://github.com/scalableminds/webknossos/pull/8806)
+ The view to change one’s email address is now available in the top-right dropdown menu of the navbar. [#8806](https://github.com/scalableminds/webknossos/pull/8806)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - The view to change ones email address is now available in the top right dropdown menu of the navbar. [#8806](https://github.com/scalableminds/webknossos/pull/8806) | |
| - The view to change one’s email address is now available in the top-right dropdown menu of the navbar. [#8806](https://github.com/scalableminds/webknossos/pull/8806) |
🤖 Prompt for AI Agents
In CHANGELOG.released.md around line 55 the phrase uses the non-possessive
"ones" instead of the correct possessive "one's"; update the line to use "one's"
(e.g., "The view to change one's email address is now available in the top right
dropdown menu of the navbar.
[#8806](https://github.com/scalableminds/webknossos/pull/8806)").
| - Fixed viewing a dataset zoomed far out which has rotations configured in the dataset settings Data Source tab. [#8819](https://github.com/scalableminds/webknossos/pull/8819) | ||
| - Fixed that the floodfilling tool in 2D mode and without a bounding box restriction was only filling a slice of the selected area. [#8828](https://github.com/scalableminds/webknossos/pull/8828) | ||
| - Fixed a bug that impacted some proofreading interactions when e.g. splitting an agglomerate. [#8830](https://github.com/scalableminds/webknossos/pull/8830) | ||
| - Fixed a bug where hdf5 attachments explicitly mentioned in the datasource-properties.json would not be readable if they have an absolute path but no `file://` prefix. [#8832](https://github.com/scalableminds/webknossos/pull/8832) |
There was a problem hiding this comment.
Capitalization: HDF5.
- ... where hdf5 attachments explicitly mentioned ...
+ ... where HDF5 attachments explicitly mentioned ...📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - Fixed a bug where hdf5 attachments explicitly mentioned in the datasource-properties.json would not be readable if they have an absolute path but no `file://` prefix. [#8832](https://github.com/scalableminds/webknossos/pull/8832) | |
| - Fixed a bug where HDF5 attachments explicitly mentioned in the datasource-properties.json would not be readable if they have an absolute path but no `file://` prefix. [#8832](https://github.com/scalableminds/webknossos/pull/8832) |
🤖 Prompt for AI Agents
In CHANGELOG.released.md around line 60, the term "hdf5" is incorrectly
lowercased; update the sentence to use the correct capitalization "HDF5" (e.g.,
"HDF5 attachments") so the changelog reads with proper acronym casing.
Comment on lines
+62
to
+63
| - Fix loading segment statistics like segment volume and segment bounding box for N-Dimensional datasets and annotations. [#8837](https://github.com/scalableminds/webknossos/pull/8837) | ||
| - WK crashing when changing the color of an empty segment group. [#8868](https://github.com/scalableminds/webknossos/pull/8868) |
There was a problem hiding this comment.
Tense consistency and sentence completion.
- “Fix loading …” → “Fixed loading …”
- Make Line 63 a complete sentence.
- Fix loading segment statistics like segment volume and segment bounding box for N-Dimensional datasets and annotations. [#8837](https://github.com/scalableminds/webknossos/pull/8837)
- WK crashing when changing the color of an empty segment group. [#8868](https://github.com/scalableminds/webknossos/pull/8868)
+ Fixed loading segment statistics (e.g., segment volume and segment bounding box) for N‑dimensional datasets and annotations. [#8837](https://github.com/scalableminds/webknossos/pull/8837)
+ Fixed a crash in WEBKNOSSOS when changing the color of an empty segment group. [#8868](https://github.com/scalableminds/webknossos/pull/8868)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - Fix loading segment statistics like segment volume and segment bounding box for N-Dimensional datasets and annotations. [#8837](https://github.com/scalableminds/webknossos/pull/8837) | |
| - WK crashing when changing the color of an empty segment group. [#8868](https://github.com/scalableminds/webknossos/pull/8868) | |
| - Fixed loading segment statistics (e.g., segment volume and segment bounding box) for N-dimensional datasets and annotations. [#8837](https://github.com/scalableminds/webknossos/pull/8837) | |
| - Fixed a crash in WEBKNOSSOS when changing the color of an empty segment group. [#8868](https://github.com/scalableminds/webknossos/pull/8868) |
🤖 Prompt for AI Agents
In CHANGELOG.released.md around lines 62 to 63, ensure tense consistency and
complete sentences by changing the bullet on line 62 from “Fix loading segment
statistics like segment volume and segment bounding box for N-Dimensional
datasets and annotations.” to “Fixed loading segment statistics such as segment
volume and segment bounding box for N‑Dimensional datasets and annotations.” and
change line 63 from “WK crashing when changing the color of an empty segment
group.” to a complete past‑tense statement such as “Fixed WK crash when changing
the color of an empty segment group.” to match style and tense.
| - WK crashing when changing the color of an empty segment group. [#8868](https://github.com/scalableminds/webknossos/pull/8868) | ||
|
|
||
| ### Removed | ||
| - An administrator of the organization can no longer update a users email address. This has to be done by the user themselves. [#8671](https://github.com/scalableminds/webknossos/pull/8671) |
There was a problem hiding this comment.
Grammar: possessive “user’s”.
- An administrator of the organization can no longer update a users email address. This has to be done by the user themselves. [#8671](https://github.com/scalableminds/webknossos/pull/8671)
+ An administrator of the organization can no longer update a user’s email address. This has to be done by the user themselves. [#8671](https://github.com/scalableminds/webknossos/pull/8671)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - An administrator of the organization can no longer update a users email address. This has to be done by the user themselves. [#8671](https://github.com/scalableminds/webknossos/pull/8671) | |
| - An administrator of the organization can no longer update a user’s email address. This has to be done by the user themselves. [#8671](https://github.com/scalableminds/webknossos/pull/8671) |
🤖 Prompt for AI Agents
In CHANGELOG.released.md around line 66, the phrase "a users email address" uses
a non-possessive plural form; change it to the possessive "a user's email
address" so the sentence reads: "An administrator of the organization can no
longer update a user's email address. This has to be done by the user
themselves. [#8671](https://github.com/scalableminds/webknossos/pull/8671)".
normanrz
approved these changes
Aug 27, 2025
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
tools/hosting/docker-compose.yml (1)
5-5: Optional: pin by digest and document the default tag.For reproducibility, consider pinning release images by digest (tag + @sha256) and adding DOCKER_TAG=25.09.0 to the hosting README/.env example to keep defaults discoverable.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
tools/hosting/docker-compose.yml(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: MichaelBuessemeyer
PR: scalableminds/webknossos#8221
File: CHANGELOG.unreleased.md:14-14
Timestamp: 2024-11-25T14:52:23.201Z
Learning: For the `webknossos` project, changelog entries in `CHANGELOG.unreleased.md` should be concise and avoid unnecessary detailed explanations.
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: build-smoketest-push
- GitHub Check: backend-tests
- GitHub Check: frontend-tests
🔇 Additional comments (2)
tools/hosting/docker-compose.yml (2)
5-5: Release image bump looks good.Defaulting to 25.09.0 via
${DOCKER_TAG:-...}aligns with the release and preserves override behavior.
5-5: Docker image tags verifiedBoth
scalableminds/webknossos:25.09.0andscalableminds/fossildb:master__510are confirmed published on Docker Hub. No further action required—safe to merge.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.