Merge branch 'master' of github.com:scalableminds/webknossos into account-settings-page

Author: hotzenklotz

.github/PULL_REQUEST_TEMPLATE.md

@@ -12,8 +12,8 @@
 
 ------
 (Please delete unneeded items, merge only when none are left open)
-- [ ] Updated [changelog](../blob/master/CHANGELOG.unreleased.md#unreleased)
-- [ ] Updated [migration guide](../blob/master/MIGRATIONS.unreleased.md#unreleased) if applicable
+- [ ] Added changelog entry (create a `$PR_NUMBER.md` file in `unreleased_changes` or use `./tools/create-changelog-entry.py`)
+- [ ] Added migration guide entry if applicable (edit the same file as for the changelog)
 - [ ] Updated [documentation](../blob/master/docs) if applicable
 - [ ] Adapted [wk-libs python client](https://github.com/scalableminds/webknossos-libs/tree/master/webknossos/webknossos/client) if relevant API parts change
 - [ ] Removed dev-only changes like prints and application.conf edits

CHANGELOG.released.md

@@ -7,6 +7,12 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Calendar Versioning](http://calver.org/) `0Y.0M.MICRO`.
 For upgrade instructions, please check the [migration guide](MIGRATIONS.released.md).
 
+## [25.06.2](https://github.com/scalableminds/webknossos/releases/tag/25.06.2) - 2025-06-17
+[Commits](https://github.com/scalableminds/webknossos/compare/25.06.1...25.06.2)
+
+### Fixed
+- Fixed a regression that led to incorrect behavior when trying to jump to the last branchpoint even though no branchpoint existed. [#8695](https://github.com/scalableminds/webknossos/pull/8695)
+
 ## [25.06.1](https://github.com/scalableminds/webknossos/releases/tag/25.06.1) - 2025-06-03
 [Commits](https://github.com/scalableminds/webknossos/compare/25.06.0...25.06.1)
 

CHANGELOG.unreleased.md

@@ -1,29 +1 @@
-# Changelog (Unreleased)
-
-All notable (yet unreleased) user-facing changes to WEBKNOSSOS are documented in this file.
-See `CHANGELOG.released.md` for the changes which are part of official releases.
-
-The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
-and this project adheres to [Calendar Versioning](http://calver.org/) `0Y.0M.MICRO`.
-For upgrade instructions, please check the [migration guide](MIGRATIONS.released.md).
-
-## Unreleased
-[Commits](https://github.com/scalableminds/webknossos/compare/25.06.1...HEAD)
-
-### Added
-- In shared annotations with multiple authors, some changes are now stored per user. This means that other users won’t see all those changes if their own diverge. This includes the current position and zoom, visibilities of trees, bounding boxes, and segments (as specified with the checkboxes in the lists), as well as which groups are expanded in the lists. The annotation owner’s user state is used as a fallback for users who haven’t explicitly changed these values themselves. [#8542](https://github.com/scalableminds/webknossos/pull/8542)
-- Added the ability to duplicate trees in skeleton annotations. Users can create a copy of any tree (including all nodes, edges, and properties) via the context menu in the skeleton tab. [#8662](https://github.com/scalableminds/webknossos/pull/8662)
-- Meshes are now reloaded using their previous opacity value. [#8622](https://github.com/scalableminds/webknossos/pull/8622)
-
-### Changed
-
-### Fixed
-- Improved efficiency of saving bounding box related changes. [#8492](https://github.com/scalableminds/webknossos/pull/8492)
-- When deleting a dataset, its caches are cleared, so that if a new dataset by the same name is uploaded afterwards, only new data is loaded. [#8638](https://github.com/scalableminds/webknossos/pull/8638)
-- Fixed the contrast of the WelcomeToast buttons. Updated `antd` to version `5.22`.[#8688](https://github.com/scalableminds/webknossos/pull/8688)
-- Fixed a race condition when starting proofreading with a split action. [#8676](https://github.com/scalableminds/webknossos/pull/8676)
-- Fixed that activating a mapping got stuck when a dataset was opened in "view" mode. [#8687](https://github.com/scalableminds/webknossos/pull/8687)
-
-### Removed
-
-### Breaking Changes
+WARNING: This file will be deleted in the next release. If you have a merge conflict in this file, please ensure that the file only contains this warning text after the merge. Please execute `./tools/create-changelog-entry.py` and move the changelog for your PR into the file that was created by the script. From now on, this will be the new way of adding changelog and migration guide entries for a PR. The created files will be aggregated when a release is created.

MIGRATIONS.released.md

@@ -6,6 +6,12 @@ See `MIGRATIONS.unreleased.md` for the changes which are not yet part of an offi
 This project adheres to [Calendar Versioning](http://calver.org/) `0Y.0M.MICRO`.
 User-facing changes are documented in the [changelog](CHANGELOG.released.md).
 
+## [25.06.2](https://github.com/scalableminds/webknossos/releases/tag/25.06.2) - 2025-06-17
+[Commits](https://github.com/scalableminds/webknossos/compare/25.06.1...25.06.2)
+
+### Postgres Evolutions:
+None.
+
 ## [25.06.1](https://github.com/scalableminds/webknossos/releases/tag/25.06.1) - 2025-06-03
 [Commits](https://github.com/scalableminds/webknossos/compare/25.06.0...25.06.1)
 

MIGRATIONS.unreleased.md

@@ -1,14 +1 @@
-# Migration Guide (Unreleased)
-All migrations (for unreleased versions) of WEBKNOSSOS are documented in this file.
-See `MIGRATIONS.released.md` for the migrations which are part of official releases.
-
-This project adheres to [Calendar Versioning](http://calver.org/) `0Y.0M.MICRO`.
-User-facing changes are documented in the [changelog](CHANGELOG.released.md).
-
-## Unreleased
-[Commits](https://github.com/scalableminds/webknossos/compare/25.06.1...HEAD)
-
-- The default thread pool size was increased from 5 to 10 times the number of available CPUs (capped at 1000). Note that wk may need slightly more memory because of this. [#8686](https://github.com/scalableminds/webknossos/pull/8686)
-
-### Postgres Evolutions:
-- [134-dataset-layer-attachments.sql](conf/evolutions/134-dataset-layer-attachments.sql)
+WARNING: This file will be deleted in the next release. If you have a merge conflict in this file, please ensure that the file only contains this warning text after the merge. Please execute `./tools/create-changelog-entry.py` and move the changelog for your PR into the file that was created by the script. From now on, this will be the new way of adding changelog and migration guide entries for a PR. The created files will be aggregated when a release is created.

app/controllers/AuthenticationController.scala

@@ -215,7 +215,7 @@ class AuthenticationController @Inject()(
     } yield result
 
   def accessibleBySwitching(datasetId: Option[ObjectId],
-                            annotationId: Option[String],
+                            annotationId: Option[ObjectId],
                             workflowHash: Option[String]): Action[AnyContent] = sil.SecuredAction.async {
     implicit request =>
       for {

app/controllers/WKRemoteTracingStoreController.scala

@@ -82,8 +82,7 @@ class WKRemoteTracingStoreController @Inject()(tracingStoreService: TracingStore
       tracingStoreService.validateAccess(name, key) { _ =>
         val report = request.body
         for {
-          annotationId <- ObjectId.fromString(report.annotationId)
-          annotation <- annotationDAO.findOne(annotationId)
+          annotation <- annotationDAO.findOne(report.annotationId)
           _ <- ensureAnnotationNotFinished(annotation)
           _ <- annotationDAO.updateModified(annotation._id, Instant.now)
           _ = report.statistics.map(statistics => annotationService.updateStatistics(annotation._id, statistics))

app/models/annotation/AnnotationLayerPrecedence.scala

@@ -29,7 +29,7 @@ case class RedundantTracingProperties(
     zoomLevel: Double,
     userBoundingBoxes: Seq[NamedBoundingBoxProto],
     editPositionAdditionalCoordinates: Seq[AdditionalCoordinateProto],
-    userStateBoundingBoxVisibilities: Map[String, Seq[Id32WithBool]] // UserId → Seq(bboxId, bboxIsVisible)
+    userStateBoundingBoxVisibilities: Map[ObjectId, Seq[Id32WithBool]] // UserId → Seq(bboxId, bboxIsVisible)
 )
 
 trait AnnotationLayerPrecedence extends FoxImplicits {
@@ -79,7 +79,7 @@ trait AnnotationLayerPrecedence extends FoxImplicits {
       userStates: Seq[SkeletonUserStateProto],
       oldPrecedenceLayerProperties: RedundantTracingProperties): Seq[SkeletonUserStateProto] = {
     val adaptedExistingUserStates = userStates.map { userState =>
-      val userId = userState.userId
+      val userId = ObjectId(userState.userId)
       oldPrecedenceLayerProperties.userStateBoundingBoxVisibilities.get(userId) match {
         case None => userState
         case Some(precedenceBboxVisibilities) =>
@@ -88,9 +88,9 @@ trait AnnotationLayerPrecedence extends FoxImplicits {
     }
     // We also have to create new user states for the users the old precedence layer has, but the new precedence layer is missing.
     val newUserPrecedenceProperties = oldPrecedenceLayerProperties.userStateBoundingBoxVisibilities.filter(tuple =>
-      !userStates.exists(_.userId == tuple._1))
+      !userStates.exists(_.userId == tuple._1.toString))
     val newUserStates = newUserPrecedenceProperties.map {
-      case (userId: String, boundingBoxVisibilities: Seq[Id32WithBool]) =>
+      case (userId: ObjectId, boundingBoxVisibilities: Seq[Id32WithBool]) =>
         SkeletonTracingDefaults
           .emptyUserState(userId)
           .copy(
@@ -104,7 +104,7 @@ trait AnnotationLayerPrecedence extends FoxImplicits {
       userStates: Seq[VolumeUserStateProto],
       oldPrecedenceLayerProperties: RedundantTracingProperties): Seq[VolumeUserStateProto] = {
     val adaptedExistingUserStates = userStates.map { userState =>
-      val userId = userState.userId
+      val userId = ObjectId(userState.userId)
       oldPrecedenceLayerProperties.userStateBoundingBoxVisibilities.get(userId) match {
         case None => userState
         case Some(precedenceBboxVisibilities) =>
@@ -113,9 +113,9 @@ trait AnnotationLayerPrecedence extends FoxImplicits {
     }
     // We also have to create new user states for the users the old precedence layer has, but the new precedence layer is missing.
     val newUserPrecedenceProperties = oldPrecedenceLayerProperties.userStateBoundingBoxVisibilities.filter(tuple =>
-      !userStates.exists(_.userId == tuple._1))
+      !userStates.exists(_.userId == tuple._1.toString))
     val newUserStates = newUserPrecedenceProperties.map {
-      case (userId: String, boundingBoxVisibilities: Seq[Id32WithBool]) =>
+      case (userId: ObjectId, boundingBoxVisibilities: Seq[Id32WithBool]) =>
         VolumeTracingDefaults
           .emptyUserState(userId)
           .copy(
@@ -194,7 +194,7 @@ trait AnnotationLayerPrecedence extends FoxImplicits {
           s.userBoundingBoxes ++ s.userBoundingBox.map(
             com.scalableminds.webknossos.datastore.geometry.NamedBoundingBoxProto(0, None, None, None, _)),
           s.editPositionAdditionalCoordinates,
-          s.userStates.map(userState => (userState.userId, userState.boundingBoxVisibilities)).toMap
+          s.userStates.map(userState => (ObjectId(userState.userId), userState.boundingBoxVisibilities)).toMap
         )
       case Right(v) =>
         RedundantTracingProperties(
@@ -204,7 +204,7 @@ trait AnnotationLayerPrecedence extends FoxImplicits {
           v.userBoundingBoxes ++ v.userBoundingBox.map(
             com.scalableminds.webknossos.datastore.geometry.NamedBoundingBoxProto(0, None, None, None, _)),
           v.editPositionAdditionalCoordinates,
-          v.userStates.map(userState => (userState.userId, userState.boundingBoxVisibilities)).toMap
+          v.userStates.map(userState => (ObjectId(userState.userId), userState.boundingBoxVisibilities)).toMap
         )
     }
 }

app/models/annotation/AnnotationMerger.scala

@@ -74,7 +74,7 @@ class AnnotationMerger @Inject()(datasetDAO: DatasetDAO, tracingStoreService: Tr
     for {
       dataset <- datasetDAO.findOne(datasetId)
       tracingStoreClient: WKRemoteTracingStoreClient <- tracingStoreService.clientFor(dataset)
-      mergedAnnotationProto <- tracingStoreClient.mergeAnnotationsByIds(annotations.map(_.id),
+      mergedAnnotationProto <- tracingStoreClient.mergeAnnotationsByIds(annotations.map(_._id),
                                                                         annotations.map(_._user),
                                                                         newAnnotationId,
                                                                         toTemporaryStore,

app/models/annotation/AnnotationService.scala

@@ -801,7 +801,7 @@ class AnnotationService @Inject()(
       dataStore <- dataStoreDAO.findOneByName(dataset._dataStore.trim) ?~> "datastore.notFound"
       tracingStore <- tracingStoreDAO.findFirst
       annotationSource = AnnotationSource(
-        id = annotation.id,
+        id = annotation._id,
         annotationLayers = annotation.annotationLayers,
         datasetDirectoryName = dataset.directoryName,
         organizationId = organization._id,