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

Add slew_time function #63

Merged
merged 1 commit into from
May 8, 2021
Merged

Add slew_time function #63

merged 1 commit into from
May 8, 2021

Conversation

lpsinger
Copy link
Contributor

@lpsinger lpsinger commented May 8, 2021

This is not yet used anywhere, but we will need it to implement slew time constraints.

This is not yet used anywhere, but we will need it to implement
slew time constraints.
@lpsinger lpsinger requested a review from mcoughlin May 8, 2021 14:56
@codecov
Copy link

codecov bot commented May 8, 2021

Codecov Report

Merging #63 (e791995) into main (67eb066) will increase coverage by 0.10%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
+ Coverage   32.05%   32.15%   +0.10%     
==========================================
  Files          29       30       +1     
  Lines        1357     1362       +5     
==========================================
+ Hits          435      438       +3     
- Misses        922      924       +2     
Impacted Files Coverage Δ
dorado/scheduling/_slew.py 50.00% <50.00%> (ø)
dorado/scheduling/__init__.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 67eb066...e791995. Read the comment docs.

Copy link
Collaborator

@mcoughlin mcoughlin left a comment

Choose a reason for hiding this comment

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

Yes good to have this in here for dev purposes.

@mcoughlin mcoughlin merged commit 8d2e373 into nasa:main May 8, 2021
@lpsinger lpsinger deleted the slew-time branch May 9, 2021 13:48
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.

2 participants