Skip to content
This repository has been archived by the owner on Dec 16, 2024. It is now read-only.

Updates and bug fixes #163

Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

Updates and bug fixes #163

wants to merge 31 commits into from

Conversation

Stoops-ML
Copy link
Collaborator

@Stoops-ML Stoops-ML commented Dec 9, 2024

  • Bumps to v2.1.0
  • Updates readme
  • Sequence -> TimeIntervalCollection
  • Add validations on many properties
  • Correct usage of Position, PositionListandPositionListOfList` (according to the czml-writer wiki)
  • Type classes cannot be initialised to None
  • Fix usage of SolidColorMaterial in Ellipsoid
  • Fix usage of 'solidColorinMaterial`
  • Fix usage of 'polyOutlineinMaterial`
  • All properties can be TimeIntervalCollection
  • Add Cartesian3VelocityValue type
  • Expand testing
  • ReferenceValue.string -> ReferenceValue.value
  • RgbafValue and RgbaValue checks time values

Daniel Stoops added 27 commits December 2, 2024 16:40
…onvert all lists in Position and PositionLists to correct types
…ex colors, Color() converts lists to their corresponding value
…stOfListsValue, ReferenceListValue, ReferenceListOfListsValue, Cartesian3ListOfListsValue
Copy link

codecov bot commented Dec 9, 2024

Codecov Report

Attention: Patch coverage is 99.65577% with 2 lines in your changes missing coverage. Please review.

Project coverage is 99.53%. Comparing base (993f614) to head (406ca83).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/czml3/properties.py 99.52% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #163      +/-   ##
==========================================
+ Coverage   99.44%   99.53%   +0.08%     
==========================================
  Files          11       11              
  Lines         898     1064     +166     
==========================================
+ Hits          893     1059     +166     
  Misses          5        5              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Stoops-ML Stoops-ML enabled auto-merge December 9, 2024 15:47
@Stoops-ML
Copy link
Collaborator Author

@astrojuanlu bump

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant