-
Notifications
You must be signed in to change notification settings - Fork 29
Release 25.03.0 #8491
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.
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
Release 25.03.0 #8491
Conversation
* rename some variables * more renaming * more renaming * fix undo in merger mode * format * change any to number --------- Co-authored-by: Michael Büßemeyer <frameworklinux+MichaelBuessemeyer@users.noreply.github.com> Co-authored-by: MichaelBuessemeyer <39529669+MichaelBuessemeyer@users.noreply.github.com>
* fix highlighting the correct navbar item * update changelog
Warning Rate limit exceeded@philippotto has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 9 minutes and 45 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis pull request introduces updates across documentation, migrations, and frontend code. Datasets now support additional data types, and enhancements have been made to navigation, task creation, and annotation handling. Database migrations are updated with new requirements and SQL files, and the frontend components have been refactored—switching navigation hooks in the Navbar and revising the merger mode logic to use unmapped segment IDs. Several bug fixes and variable/method renamings ensure consistency across the codebase. Changes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
||
### Added | ||
- Added support for datasets with the following data types: int8, int16, int32, uint32 (support for color was added, support for segmentation already existed before) and int64 (segmentation only). [#8325](https://github.com/scalableminds/webknossos/pull/8325) | ||
- Added a command palette that allows navigating between pages, switching tools and accessing some user settings via Ctrl+P. [#8447](https://github.com/scalableminds/webknossos/pull/8447/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’d call this a highlight ;-) Or aren’t we doing those anymore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, I forgot about the highlights!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
CHANGELOG.released.md (1)
24-24
: Minor inconsistency in URL formatting.There's a trailing slash at the end of the URL that isn't consistent with most other PR links in the changelog. This isn't a critical issue, as some other entries also have trailing slashes, but for consistency, consider removing trailing slashes in future entries.
- When using a zarr link to a wk-served data layer as another layer's source, the user's token is used to access the data. [#8322](https://github.com/scalableminds/webknossos/pull/8322/) + When using a zarr link to a wk-served data layer as another layer's source, the user's token is used to access the data. [#8322](https://github.com/scalableminds/webknossos/pull/8322)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
CHANGELOG.released.md
(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: circleci_build
🔇 Additional comments (2)
CHANGELOG.released.md (2)
13-15
: Looking good! Highlights section properly included.The highlights section has been appropriately added with significant features. This addresses previous feedback (from the comment thread) about including highlights in the changelog.
10-46
: Well-structured changelog entry for version 25.03.0.The new changelog entry is well-formatted and comprehensive, providing clear information about new features, changes, and fixes in this release. The content follows the established pattern of the changelog with appropriate sections and formatting.
Based on 38d92ac. Additionally, cherry picked:
Once approved, I will create a git tag and then pull the master in.