Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Type of change
Jira ticket 📘
https://cognitedata.atlassian.net/browse/
Description 📝
I think this is a simple fix that will potentially reduce waste in loading and then discarding sectors. I have not extensively tested this, but it seems like a easy win for improved loading performance.
I know this fix is not as important with remote models as the warning
Abort signal is not supported for local models
appears in the log and I presume this would have cancelled something.How has this been tested? 🔍
Tested with local model in chrome using dev-tools to throttle network latency to 200ms. Also tested with remote Equinor-models, with improvements when the network or device is slow. Combined with #3674 I believe this is a big difference in performance.
Lots of more possible improvements (should cancel the ongoing batch for instance) but this is a oneliner, so hard to beat.
Test instructions ℹ️
I suggest playing around with the code to see the impact
for example this debug code is good to show the loading of old sectors still active way later.
Checklist ☑️