Skip to content

Commit

Permalink
Merge pull request #1368 from xeokit/xeolabs-patch-4
Browse files Browse the repository at this point in the history
[FIX] Ensure DTX-enabled SceneModel still uses VBOs for textures
  • Loading branch information
xeolabs authored Feb 13, 2024
2 parents 387efcf + f4a29d8 commit 2620b1c
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/viewer/scene/model/SceneModel.js
Original file line number Diff line number Diff line change
Expand Up @@ -2779,7 +2779,8 @@ export class SceneModel extends Component {

const useDTX = (!!this._dtxEnabled && (cfg.primitive === "triangles"
|| cfg.primitive === "solid"
|| cfg.primitive === "surface"));
|| cfg.primitive === "surface"))
&& (!cfg.textureSetId);

cfg.origin = cfg.origin ? math.addVec3(this._origin, cfg.origin, math.vec3()) : this._origin;

Expand Down Expand Up @@ -2997,7 +2998,8 @@ export class SceneModel extends Component {
const useDTX = (!!this._dtxEnabled
&& (cfg.geometry.primitive === "triangles"
|| cfg.geometry.primitive === "solid"
|| cfg.geometry.primitive === "surface"));
|| cfg.geometry.primitive === "surface"))
&& (!cfg.textureSetId);

if (useDTX) {

Expand Down

0 comments on commit 2620b1c

Please sign in to comment.