search

nasa / dorado-scheduling

Dorado observation planning and scheduling simulations
Other
22 stars 8 forks source link

Updating UVEX TLE #71

Closed bwgref closed 3 years ago

bwgref commented 3 years ago

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

codecov[bot] commented 3 years ago

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%) :arrow_down:
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 commented 3 years ago

Waiting on workflow approval to rerun checks

mcoughlin commented 3 years ago

Unit tests pass!

lpsinger commented 3 years ago

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

mcoughlin commented 3 years ago

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

lpsinger commented 3 years ago

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