IHE-RO-Tooling/dicom-browser
IHE-RO-Tooling/dicom-browser
1
0
Fork 0
Code Issues 4 Pull requests Projects Releases 2 Packages Wiki Activity Actions

fix: skip eager series query when instance count already known #93

Merged
christof.schadt merged 2 commits from fix-eager-series-query into main 2026-07-13 09:06:19 +02:00
Conversation 0 Commits 2 Files changed 1 +13 -2
christof.schadt commented 2026-07-13 09:04:31 +02:00
Owner
Copy link

The auto-query loop retrieved instances for every not-yet-loaded series
just to display the instance count. When the series-level query already
reports NumberOfSeriesRelatedInstances (0020,1209) the tree uses that
count directly, so the retrieval is wasted traffic — and on a
DICOMweb->DIMSE gateway each such query escalates to a per-series
retrieval. Skip series that already have a reported count.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

The auto-query loop retrieved instances for every not-yet-loaded series just to display the instance count. When the series-level query already reports NumberOfSeriesRelatedInstances (0020,1209) the tree uses that count directly, so the retrieval is wasted traffic — and on a DICOMweb->DIMSE gateway each such query escalates to a per-series retrieval. Skip series that already have a reported count. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
fix: skip eager series query when instance count already known
Some checks failed
fmt / rustfmt (push) Failing after 43s
Details
layering / check-layering (push) Successful in 9s
Details
284ea341e6 
The auto-query loop retrieved instances for every not-yet-loaded series
just to display the instance count. When the series-level query already
reports NumberOfSeriesRelatedInstances (0020,1209) the tree uses that
count directly, so the retrieval is wasted traffic — and on a
DICOMweb->DIMSE gateway each such query escalates to a per-series
retrieval. Skip series that already have a reported count.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
christof.schadt deleted branch fix-eager-series-query 2026-07-13 09:08:22 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
wontfix
This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
IHE-RO-Tooling/dicom-browser!93
No description provided.