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

CELE-109 Fixes screen flash with 3D viewer #67

Merged
merged 1 commit into from
Oct 23, 2024
Merged

Conversation

aranega
Copy link
Member

@aranega aranega commented Oct 23, 2024

The problem of flash came from the fact that, depending on the dataset, the data couldn't be fetch. When that was the case, none of the scene was rendered, even if only one file couldn't be loaded. This commit changes the way the 3D images are loaded, and shows an error message to inform the user if one of the 3D images cannot be displayed (the other images for the requested neurons are displayed if they exists in the bucket).

The problem of flash came from the fact that, depending on the dataset,
the data couldn't be fetch. When that was the case, none of the scene
was rendered, even if only one file couldn't be loaded. This commit
changes the way the 3D images are loaded, and shows an error message to
inform the user if one of the 3D images cannot be displayed (the other
images for the requested neurons are displayed if they exists in the
bucket).
Copy link
Member

@ddelpiano ddelpiano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ddelpiano ddelpiano merged commit 1a2ae58 into develop Oct 23, 2024
10 checks passed
@ddelpiano ddelpiano deleted the feature/CELE-109 branch October 23, 2024 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants