changeset 2054:70b79788c2f3

clarified TODO wrt /metadata
author Alain Mazy <am@osimis.io>
date Thu, 30 Mar 2023 17:34:07 +0200
parents d77fea5934fb
children f7cbc58ff44d 1096d92febfc
files TODO
diffstat 1 files changed, 6 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/TODO	Thu Mar 30 17:20:01 2023 +0200
+++ b/TODO	Thu Mar 30 17:34:07 2023 +0200
@@ -57,7 +57,12 @@
 
 * Replace calls to /studies/.../series/.../metadata by calls to
   /instances?StudyInstanceUID=...&SeriesInstanceUID=...&includefield=...&includefield=...
-  to benefit from ExtraMainDicomTags stored in DB and avoid reading from disk.
+  to benefit from ExtraMainDicomTags stored in DB and avoid reading from disk when not necessary.
+  This way, we request only the tags that the viewer is going to use.
+  The /metadata requests all tags and therefore will read from the storage unless you have
+  specified "DicomWeb.SeriesMetadata": "MainDicomTags"; this is an acceptable workaround
+  for now but the first approach is cleaner and removes dependencies between the DicomWeb config
+  and stone.  It might also be more efficient on other backends as well.
 
 --------
 Wishlist