-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[Feature Request] Study prefetching service and progress bar for display sets #3363
Comments
Same as #883 , pending since long time |
Similar to issue #883, this pending task has been awaiting completion for an extended duration. |
Me also facing the same issue. Needs to get fixed ASAP. Same as #883 |
Same here, eagerly await this feature! |
|
@dan-rukas any updates on this one? |
Someone from community commented they would like to have a load time for the study which makes sense in this prefetcher |
We will investigate and design during upcoming UI redesign |
@james-hanks any updates on this? |
@fedorov this in our queue for next major release (3.8). We are currently in design review phase. We will update the ticket with design when it is finalized - please feel free to take a look and comment at that time |
|
Is this feature being worked on actively? Maybe someone knows a branch with corresponding changes ongoing work/drafts? |
Any update on this? |
Not exactly a prefetching service as requested by this ticket but something I figured could be used for custom preloading of certain images (like all from specific displaySet) and potentially adapted to load all display sets in a background.
|
No update, but this will come for next release |
@sedghi When we can expect next release ? |
@PatrykLuszcz-1 official release will be somewhere in September probably, but the feature will get merged into master branch (dev) sooner see our branching logic |
@lscoder will start on this this week. We will target the top-bottom for now |
Any updates on this? I tried to run @lscoder PR to test it, but didn't get what's changed and how it should work. |
@chriszao you should wait for Leo's PR to get merged first |
@chriszao The changes are merged and you need to enable the service first because it is disabled by default. You need to add something like this to your config file: studyPrefetcher: {
enabled: true,
displaySetCount: 2,
maxNumPrefetchRequests: 10,
order: 'closest',
}, You can set the order to "closest", "downward" or "upward". |
@lscoder You saved my life with that PR hahaha! 🎉. |
@chriszao Good to hear that lol It downloads any StudyPrefetcher.-.Related.Studies.mp4Loading all the displaySets from related studies would not be a major change but it would require changing how the prefetcher works because it does not load studies but displaySets from any study. |
Request
Context
The status of series loading is missing. Frame fetching/caching behavior is unclear - even though I scroll through a series several times, the scrolling remains jumpy, and the “Loading…” message keeps showing up. How can I know when the entire series is actually loaded?
v2 Port Redesign:
@dan-rukas add design here when finalized
The text was updated successfully, but these errors were encountered: