Skip to content
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

Potential Bug in DataApi.getDataForBoundingBox leading to the returned promise to be never resolved #7740

Open
1 task done
MichaelBuessemeyer opened this issue Apr 9, 2024 · 0 comments

Comments

@MichaelBuessemeyer
Copy link
Contributor

MichaelBuessemeyer commented Apr 9, 2024

Context

During debugging the Skeleton SAM PR we noticed that in some cases the DataApi.getDataForBoundingBox API method does not return although all data for the requested bbox should be loaded already. This always included rather large bounding boxes: 1024x1024x??.

This bug needs further investigation into what is causing this.

Expected Behavior

The method should return a promise that will eventually be resolved in a reasonable amount of time. Or else an error indicating the bounding box is too large should be thrown.

Current Behavior

Sometimes the return promise is never resolved and waiting for some bucket to be loaded forever. There seemed to be a race condition here causing the bug to not be reproducible in a reliable way.

Steps to Reproduce the bug

  • Cannot reproduce the bug anymore / needs deeper investigation.

Your Environment for bug

  • Browser name and version: Chrome (newest debian based (v123))
  • Operating System and version: e.g. Ubuntu 22.04
  • Version of WEBKNOSSOS (Release or Commit): noticed at the sam-skeleton-prediction branch
@MichaelBuessemeyer MichaelBuessemeyer changed the title Potential Bug in DataApi.getDataForBoundingBox leading to the method to be stuck forever Potential Bug in DataApi.getDataForBoundingBox leading to the returned promise to be never resolved Apr 9, 2024
@fm3 fm3 added the frontend label Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants