-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Partial ellipsoid #5995
Partial ellipsoid #5995
Changes from 11 commits
fc58ff0
81d013c
e379f89
899ed31
c3ae3ff
3ff9459
dd4c52b
8ce606a
613dc63
4e373e3
3e8f06f
18ebf67
13ca0f7
a4af472
48068df
668530e
e821b77
da14f40
d7047d1
859fea7
c74c4c8
4a081e6
aafa6fd
f52ac07
9d7541d
1aeb1ab
8532e1c
e1c0796
595fa8e
3d99a52
5a8a616
dc622fa
7fc91d9
5cb6217
941f303
9719d93
4ac7d30
c4d6401
124f9ee
78e662f
89e50b1
96414d7
852c174
4a872ed
bc8faee
32bf68f
954a618
7ddf64f
8c1cd56
49d39a9
7564548
5d68ee0
fb212dc
fce6d19
acc3c53
15c17f1
0867f9e
0265054
abe35b0
fa08985
51dbad3
8fd124a
44b1478
5a67857
48abe88
f8211e8
dae1851
9eaac4c
708cccc
de4e640
414d9d4
48a7987
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1534,6 +1534,11 @@ define([ | |
|
||
processPacketData(Boolean, ellipsoid, 'show', ellipsoidData.show, interval, sourceUri, entityCollection, query); | ||
processPacketData(Cartesian3, ellipsoid, 'radii', ellipsoidData.radii, interval, sourceUri, entityCollection, query); | ||
processPacketData(Cartesian3, ellipsoid, 'innerRadii', ellipsoidData.innerRadii, interval, sourceUri, entityCollection); | ||
processPacketData(Number, ellipsoid, 'azimuthMin', ellipsoidData.azimuthMin, interval, sourceUri, entityCollection); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Throughout, please write out We avoid abbreviations in the public API: https://github.com/AnalyticalGraphicsInc/cesium/tree/master/Documentation/Contributors/CodingGuide#naming There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also, to be consistent the rest of Cesium, make "minimum" and "maximum" the prefix instead of the suffix, e.g., |
||
processPacketData(Number, ellipsoid, 'azimuthMax', ellipsoidData.azimuthMax, interval, sourceUri, entityCollection); | ||
processPacketData(Number, ellipsoid, 'elevationMin', ellipsoidData.elevationMin, interval, sourceUri, entityCollection); | ||
processPacketData(Number, ellipsoid, 'elevationMax', ellipsoidData.elevationMax, interval, sourceUri, entityCollection); | ||
processPacketData(Boolean, ellipsoid, 'fill', ellipsoidData.fill, interval, sourceUri, entityCollection, query); | ||
processMaterialPacketData(ellipsoid, 'material', ellipsoidData.material, interval, sourceUri, entityCollection, query); | ||
processPacketData(Boolean, ellipsoid, 'outline', ellipsoidData.outline, interval, sourceUri, entityCollection, query); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this explicitly mention the new Cesium classes and CZML properties? And link to the Sandcastle example if you are able to add one?