Skip to content

Commit

Permalink
Update changelog and minor tweaks
Browse files Browse the repository at this point in the history
  • Loading branch information
j9liu committed Oct 13, 2022
1 parent 35a4340 commit 1b9d5b3
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
6 changes: 6 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
# Change Log

### 1.99 - 2022-11-01

##### Fixes :wrench:

- Fixed a bug where the scale of a `Model` was being incorrectly applied to its bounding sphere. [#10855](https://github.com/CesiumGS/cesium/pull/10855)

### 1.98.1 - 2022-10-03

- This is an npm only release to fix the improperly published 1.98.
Expand Down
6 changes: 3 additions & 3 deletions Specs/Scene/Model/ModelSpec.js
Original file line number Diff line number Diff line change
Expand Up @@ -3291,7 +3291,7 @@ describe(
);
Matrix4.getTranslation(expectedTranslation, expectedCenter);

let boundingSphere = model.boundingSphere;
const boundingSphere = model.boundingSphere;
expect(boundingSphere.center).toEqual(
Cartesian3.multiplyByScalar(
expectedCenter,
Expand All @@ -3305,12 +3305,12 @@ describe(
);

model.scale = 0.0;
boundingSphere = model.boundingSphere;
scene.renderForSpecs();
expect(boundingSphere.center).toEqual(Cartesian3.ZERO);
expect(boundingSphere.radius).toEqual(0.0);

model.scale = 1.0;
boundingSphere = model.boundingSphere;
scene.renderForSpecs();
expect(boundingSphere.center).toEqual(expectedCenter);
expect(boundingSphere.radius).toEqualEpsilon(
expectedRadius,
Expand Down

0 comments on commit 1b9d5b3

Please sign in to comment.