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

feat: return conditional solution for segment and sequences #233

Merged

Conversation

vishwa2710
Copy link
Contributor

No description provided.

@vishwa2710 vishwa2710 self-assigned this Oct 6, 2023
@vishwa2710 vishwa2710 force-pushed the users/vishwa/sequences-pass-back-condition-solution branch from 5ff075e to 41d4096 Compare October 6, 2023 18:46
@codecov
Copy link

codecov bot commented Oct 6, 2023

Codecov Report

Merging #233 (cc9ec48) into main (4465595) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #233   +/-   ##
=======================================
  Coverage   82.79%   82.79%           
=======================================
  Files          62       62           
  Lines        4743     4743           
=======================================
  Hits         3927     3927           
  Misses        816      816           
Files Coverage Δ
...nSpaceToolkit/Astrodynamics/Trajectory/Segment.hpp 100.00% <ø> (ø)
...SpaceToolkit/Astrodynamics/Trajectory/Sequence.hpp 100.00% <ø> (ø)
...aceToolkit/Astrodynamics/Trajectory/Propagator.cpp 95.17% <100.00%> (-0.07%) ⬇️
...nSpaceToolkit/Astrodynamics/Trajectory/Segment.cpp 100.00% <100.00%> (ø)
...SpaceToolkit/Astrodynamics/Trajectory/Sequence.cpp 100.00% <100.00%> (ø)

Copy link
Contributor Author

@vishwa2710 vishwa2710 left a comment

Choose a reason for hiding this comment

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

Thanks for the review!

@vishwa2710 vishwa2710 merged commit 92cbe01 into main Oct 9, 2023
12 checks passed
@vishwa2710 vishwa2710 deleted the users/vishwa/sequences-pass-back-condition-solution branch October 9, 2023 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants