Skip to content

Commit

Permalink
fix(core): reset culler state after read pixels failed
Browse files Browse the repository at this point in the history
  • Loading branch information
agviegas committed Aug 28, 2024
1 parent fdacb77 commit bded3c3
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion packages/core/package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "@thatopen/components",
"description": "Collection of core functionalities to author BIM apps.",
"version": "2.2.7",
"version": "2.2.8",
"author": "That Open Company",
"contributors": [
"Antonio Gonzalez Viegas (https://github.com/agviegas)",
Expand Down
2 changes: 1 addition & 1 deletion packages/core/src/core/Components/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ export class Components implements Disposable {
/**
* The version of the @thatopen/components library.
*/
static readonly release = "2.2.7";
static readonly release = "2.2.8";

/** {@link Disposable.onDisposed} */
readonly onDisposed = new Event<void>();
Expand Down
2 changes: 2 additions & 0 deletions packages/core/src/core/Cullers/src/culler-renderer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -189,6 +189,8 @@ export class CullerRenderer {
);
} catch (e) {
// Pixels couldn't be read, possibly because culler was disposed
this.needsUpdate = false;
this._isWorkerBusy = false;
this.renderer.setRenderTarget(null);
return;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -255,6 +255,7 @@ void main() {
} catch (e) {
// Pixels couldn't be read, possibly because culler was disposed
this.renderer.setRenderTarget(null);
this._isWorkerBusy = false;
return;
}

Expand Down

0 comments on commit bded3c3

Please sign in to comment.