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

Add additional smokescreen to build-ts #8908

Merged
merged 10 commits into from
Jun 5, 2020
1 change: 1 addition & 0 deletions .prettierignore
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
!**/*.css
!**/*.html
!**/*.md
!**/*.ts

# Re-ignore a few things caught above
**/*.min.js
Expand Down
2 changes: 2 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@
- Improved `MaterialProperty` JSDoc and TypeScript type definitions, which were missing the ability to take primitive types in addition to Property instances in their constructor.
- Fixed `EllipsoidGeodesic` JSDoc and TypeScript type definitions which incorrectly listed `result` as required.
- Fixed a bug with handling of PixelFormat's flipY. [#8893](https://github.com/CesiumGS/cesium/pull/8893)
- Fixed JSDoc and TypeScript type definitions for all `ImageryProvider` types, which were missing `defaultNightAlpha` and `defaultDayAlpha` properties.
- Fixed JSDoc and TypeScript type definitions for `Viewer` options parameter, which was incorrectly listed as required.
- Fixed a memory leak where some 3D Tiles requests were being unintentionally retained after the requests were cancelled. [#8843](https://github.com/CesiumGS/cesium/pull/8843)

### 1.70.0 - 2020-06-01
Expand Down
3 changes: 3 additions & 0 deletions Source/Core/HeadingPitchRange.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,19 +17,22 @@ function HeadingPitchRange(heading, pitch, range) {
/**
* Heading is the rotation from the local north direction where a positive angle is increasing eastward.
* @type {Number}
* @default 0.0
*/
this.heading = defaultValue(heading, 0.0);

/**
* Pitch is the rotation from the local xy-plane. Positive pitch angles
* are above the plane. Negative pitch angles are below the plane.
* @type {Number}
* @default 0.0
*/
this.pitch = defaultValue(pitch, 0.0);

/**
* Range is the distance from the center of the local frame.
* @type {Number}
* @default 0.0
*/
this.range = defaultValue(range, 0.0);
}
Expand Down
15 changes: 15 additions & 0 deletions Source/Core/HeadingPitchRoll.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,23 @@ import CesiumMath from "./Math.js";
* @param {Number} [roll=0.0] The roll component in radians.
*/
function HeadingPitchRoll(heading, pitch, roll) {
/**
* Gets or sets the heading.
* @type {Number}
* @default 0.0
*/
this.heading = defaultValue(heading, 0.0);
/**
* Gets or sets the pitch.
* @type {Number}
* @default 0.0
*/
this.pitch = defaultValue(pitch, 0.0);
/**
* Gets or sets the roll.
* @type {Number}
* @default 0.0
*/
this.roll = defaultValue(roll, 0.0);
}

Expand Down
6 changes: 3 additions & 3 deletions Source/DataSources/NodeTransformationProperty.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ var defaultNodeTransformation = new TranslationRotationScale();
* @constructor
*
* @param {Object} [options] Object with the following properties:
* @param {Property} [options.translation=Cartesian3.ZERO] A {@link Cartesian3} Property specifying the (x, y, z) translation to apply to the node.
* @param {Property} [options.rotation=Quaternion.IDENTITY] A {@link Quaternion} Property specifying the (x, y, z, w) rotation to apply to the node.
* @param {Property} [options.scale=new Cartesian3(1.0, 1.0, 1.0)] A {@link Cartesian3} Property specifying the (x, y, z) scaling to apply to the node.
* @param {Property|Cartesian3} [options.translation=Cartesian3.ZERO] A {@link Cartesian3} Property specifying the (x, y, z) translation to apply to the node.
* @param {Property|Quaternion} [options.rotation=Quaternion.IDENTITY] A {@link Quaternion} Property specifying the (x, y, z, w) rotation to apply to the node.
* @param {Property|Cartesian3} [options.scale=new Cartesian3(1.0, 1.0, 1.0)] A {@link Cartesian3} Property specifying the (x, y, z) scaling to apply to the node.
*/
function NodeTransformationProperty(options) {
options = defaultValue(options, defaultValue.EMPTY_OBJECT);
Expand Down
2 changes: 1 addition & 1 deletion Source/DataSources/PositionPropertyArray.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import ReferenceFrame from "../Core/ReferenceFrame.js";
import Property from "./Property.js";

/**
* A {@link PositionProperty} whose value is an array whose items are the computed value
* A {@link Property} whose value is an array whose items are the computed value
* of other PositionProperty instances.
*
* @alias PositionPropertyArray
Expand Down
2 changes: 1 addition & 1 deletion Source/DataSources/VelocityOrientationProperty.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ import VelocityVectorProperty from "./VelocityVectorProperty.js";
* @alias VelocityOrientationProperty
* @constructor
*
* @param {Property} [position] The position property used to compute the orientation.
* @param {PositionProperty} [position] The position property used to compute the orientation.
* @param {Ellipsoid} [ellipsoid=Ellipsoid.WGS84] The ellipsoid used to determine which way is up.
*
* @example
Expand Down
2 changes: 1 addition & 1 deletion Source/DataSources/VelocityVectorProperty.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import Property from "./Property.js";
* @alias VelocityVectorProperty
* @constructor
*
* @param {Property} [position] The position property used to compute the velocity.
* @param {PositionProperty} [position] The position property used to compute the velocity.
* @param {Boolean} [normalize=true] Whether to normalize the computed velocity vector.
*
* @example
Expand Down
32 changes: 25 additions & 7 deletions Source/Scene/ArcGisMapServerImageryProvider.js
Original file line number Diff line number Diff line change
Expand Up @@ -101,16 +101,34 @@ function ArcGisMapServerImageryProvider(options) {
* The default alpha blending value of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultAlpha = undefined;

/**
* The default alpha blending value on the night side of the globe of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number|undefined}
* @default undefined
*/
this.defaultNightAlpha = undefined;

/**
* The default alpha blending value on the day side of the globe of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number|undefined}
* @default undefined
*/
this.defaultDayAlpha = undefined;

/**
* The default brightness of this provider. 1.0 uses the unmodified imagery color. Less than 1.0
* makes the imagery darker while greater than 1.0 makes it brighter.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultBrightness = undefined;
Expand All @@ -119,15 +137,15 @@ function ArcGisMapServerImageryProvider(options) {
* The default contrast of this provider. 1.0 uses the unmodified imagery color. Less than 1.0 reduces
* the contrast while greater than 1.0 increases it.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultContrast = undefined;

/**
* The default hue of this provider in radians. 0.0 uses the unmodified imagery color.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultHue = undefined;
Expand All @@ -136,15 +154,15 @@ function ArcGisMapServerImageryProvider(options) {
* The default saturation of this provider. 1.0 uses the unmodified imagery color. Less than 1.0 reduces the
* saturation while greater than 1.0 increases it.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultSaturation = undefined;

/**
* The default gamma correction to apply to this provider. 1.0 uses the unmodified imagery color.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultGamma = undefined;
Expand Down Expand Up @@ -524,7 +542,7 @@ Object.defineProperties(ArcGisMapServerImageryProvider.prototype, {
* Gets the maximum level-of-detail that can be requested. This function should
* not be called before {@link ArcGisMapServerImageryProvider#ready} returns true.
* @memberof ArcGisMapServerImageryProvider.prototype
* @type {Number}
* @type {Number|undefined}
* @readonly
*/
maximumLevel: {
Expand Down
32 changes: 25 additions & 7 deletions Source/Scene/BingMapsImageryProvider.js
Original file line number Diff line number Diff line change
Expand Up @@ -80,16 +80,34 @@ function BingMapsImageryProvider(options) {
* The default alpha blending value of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultAlpha = undefined;

/**
* The default alpha blending value on the night side of the globe of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number|undefined}
* @default undefined
*/
this.defaultNightAlpha = undefined;

/**
* The default alpha blending value on the day side of the globe of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number|undefined}
* @default undefined
*/
this.defaultDayAlpha = undefined;

/**
* The default brightness of this provider. 1.0 uses the unmodified imagery color. Less than 1.0
* makes the imagery darker while greater than 1.0 makes it brighter.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultBrightness = undefined;
Expand All @@ -98,15 +116,15 @@ function BingMapsImageryProvider(options) {
* The default contrast of this provider. 1.0 uses the unmodified imagery color. Less than 1.0 reduces
* the contrast while greater than 1.0 increases it.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultContrast = undefined;

/**
* The default hue of this provider in radians. 0.0 uses the unmodified imagery color.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultHue = undefined;
Expand All @@ -115,15 +133,15 @@ function BingMapsImageryProvider(options) {
* The default saturation of this provider. 1.0 uses the unmodified imagery color. Less than 1.0 reduces the
* saturation while greater than 1.0 increases it.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultSaturation = undefined;

/**
* The default gamma correction to apply to this provider. 1.0 uses the unmodified imagery color.
*
* @type {Number}
* @type {Number|undefined}
* @default 1.0
*/
this.defaultGamma = 1.0;
Expand Down Expand Up @@ -404,7 +422,7 @@ Object.defineProperties(BingMapsImageryProvider.prototype, {
* Gets the maximum level-of-detail that can be requested. This function should
* not be called before {@link BingMapsImageryProvider#ready} returns true.
* @memberof BingMapsImageryProvider.prototype
* @type {Number}
* @type {Number|undefined}
* @readonly
*/
maximumLevel: {
Expand Down
32 changes: 25 additions & 7 deletions Source/Scene/GoogleEarthEnterpriseImageryProvider.js
Original file line number Diff line number Diff line change
Expand Up @@ -100,16 +100,34 @@ function GoogleEarthEnterpriseImageryProvider(options) {
* The default alpha blending value of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultAlpha = undefined;

/**
* The default alpha blending value on the night side of the globe of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number|undefined}
* @default undefined
*/
this.defaultNightAlpha = undefined;

/**
* The default alpha blending value on the day side of the globe of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number|undefined}
* @default undefined
*/
this.defaultDayAlpha = undefined;

/**
* The default brightness of this provider. 1.0 uses the unmodified imagery color. Less than 1.0
* makes the imagery darker while greater than 1.0 makes it brighter.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultBrightness = undefined;
Expand All @@ -118,15 +136,15 @@ function GoogleEarthEnterpriseImageryProvider(options) {
* The default contrast of this provider. 1.0 uses the unmodified imagery color. Less than 1.0 reduces
* the contrast while greater than 1.0 increases it.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultContrast = undefined;

/**
* The default hue of this provider in radians. 0.0 uses the unmodified imagery color.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultHue = undefined;
Expand All @@ -135,15 +153,15 @@ function GoogleEarthEnterpriseImageryProvider(options) {
* The default saturation of this provider. 1.0 uses the unmodified imagery color. Less than 1.0 reduces the
* saturation while greater than 1.0 increases it.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultSaturation = undefined;

/**
* The default gamma correction to apply to this provider. 1.0 uses the unmodified imagery color.
*
* @type {Number}
* @type {Number|undefined}
* @default undefined
*/
this.defaultGamma = undefined;
Expand Down Expand Up @@ -315,7 +333,7 @@ Object.defineProperties(GoogleEarthEnterpriseImageryProvider.prototype, {
* Gets the maximum level-of-detail that can be requested. This function should
* not be called before {@link GoogleEarthEnterpriseImageryProvider#ready} returns true.
* @memberof GoogleEarthEnterpriseImageryProvider.prototype
* @type {Number}
* @type {Number|undefined}
* @readonly
*/
maximumLevel: {
Expand Down
18 changes: 18 additions & 0 deletions Source/Scene/GoogleEarthEnterpriseMapsProvider.js
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,24 @@ function GoogleEarthEnterpriseMapsProvider(options) {
*/
this.defaultAlpha = undefined;

/**
* The default alpha blending value on the night side of the globe of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number|undefined}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure how the hell I missed these two. thanks.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Saw your comment, that's exactly why.

* @default undefined
*/
this.defaultNightAlpha = undefined;

/**
* The default alpha blending value on the day side of the globe of this provider, with 0.0 representing fully transparent and
* 1.0 representing fully opaque.
*
* @type {Number|undefined}
* @default undefined
*/
this.defaultDayAlpha = undefined;

/**
* The default brightness of this provider. 1.0 uses the unmodified imagery color. Less than 1.0
* makes the imagery darker while greater than 1.0 makes it brighter.
Expand Down
Loading