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

Updating UVEX TLE #71

Merged
merged 1 commit into from
Jun 15, 2021
Merged

Updating UVEX TLE #71

merged 1 commit into from
Jun 15, 2021

Conversation

bwgref
Copy link
Contributor

@bwgref bwgref commented Jun 14, 2021

Have not adjusted the EarthLimbConstraint in the spirit of only changing one thing at a time.

@lpsinger lpsinger self-requested a review June 14, 2021 20:43
Copy link
Contributor

@lpsinger lpsinger left a comment

Choose a reason for hiding this comment

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

Please fix the lint errors.

@codecov
Copy link

codecov bot commented Jun 14, 2021

Codecov Report

Merging #71 (3828381) into main (25ea652) will decrease coverage by 0.53%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
- Coverage   30.36%   29.83%   -0.54%     
==========================================
  Files          35       35              
  Lines        1683     1713      +30     
==========================================
  Hits          511      511              
- Misses       1172     1202      +30     
Impacted Files Coverage Δ
dorado/scheduling/mission.py 93.75% <ø> (ø)
dorado/scheduling/scripts/simsurvey.py 7.12% <0.00%> (-0.70%) ⬇️
dorado/scheduling/scripts/animate_survey.py 5.59% <0.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 25ea652...3828381. Read the comment docs.

@bwgref
Copy link
Contributor Author

bwgref commented Jun 14, 2021

Waiting on workflow approval to rerun checks

@bwgref bwgref requested a review from lpsinger June 14, 2021 21:42
dorado/scheduling/mission.py Outdated Show resolved Hide resolved
Copy link
Contributor

@lpsinger lpsinger left a comment

Choose a reason for hiding this comment

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

Still one last lint error...

Have not adjusted the EarthLimbConstraint in the spirit of only changing one thing at a time.
LINT fixes


lint again


Rewording comments


lint nazi
@mcoughlin
Copy link
Collaborator

Unit tests pass!

@mcoughlin mcoughlin merged commit 7a0f5a3 into nasa:main Jun 15, 2021
@lpsinger
Copy link
Contributor

I'm really sorry, but I have to revert this because we haven't gotten @bwgref's CLA yet. Ughhh... lawyers.

@mcoughlin
Copy link
Collaborator

So we should be using this TLE locally for now, otherwise the science teams are building search efficiencies off of unrealistic schedules no?

@lpsinger
Copy link
Contributor

I think we just need someone who has a signed CLA to provide the TLE.

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.

None yet

3 participants