Skip to content
Snippets Groups Projects
Select Git revision
  • main default protected
  • Kelvinrr-patch-3
  • radius_update
  • revert-616-apollo_pan
  • vims
  • 0.10
  • Kelvinrr-patch-2
  • revert-563-minirf_fix
  • Kelvinrr-patch-1
  • 0.9
  • acpaquette-patch-3
  • acpaquette-patch-2
  • acpaquette-patch-1
  • spiceql
  • ci-coverage
  • 0.10.0
  • 0.9.1
  • 0.9.0
  • 0.8.7
  • 0.8.8
  • 0.8.6
  • 0.8.3
  • 0.8.4
  • 0.8.5
  • 0.8.2
  • 0.8.1
  • 0.8.0
  • 0.7.3
  • 0.7.2
  • 0.7.1
  • 0.7.0
  • 0.6.5
  • 0.6.4
  • 0.6.3
  • 0.6.2
35 results

ale

  • Clone with SSH
  • Clone with HTTPS
  • Trent Hare's avatar
    Trent Hare authored
    * Update to isd_generate.py to allow for a NAIF radius override
    
    Based on this RFC, we want to change our Mars pipeline to use a sphere. https://astrodiscuss.usgs.gov/t/appl-rfc-use-iau-sphere-for-hirise-dtm-projects-in-socet-set/428
    
    
    
    Currently ALE uses the Mars elliptical defaults from the NAIF kernels. This update allows the user to override the kernels and set a spherical radius for both semimajor and semiminor in the ISD.
    
    This is not only needed for Mars. For triaxial bodies, the IAU recommends a best-fit sphere should also be set for map products. References (DOI) are listed in the updated help for the current IAU values.
    
    * remove extra file write if new radius is sent
    
    * update to allow ellipse, minor help update
    
    Add --semimajor and --semiminor parameters. I tried nargs="+" for --radius but fought for too long with argparse to do all the checking (like checking it is float). I am fine with these options but happy to hear better ways to use argparse.
    
    * Update isd_generate.py (#619)
    
    add in PROJ like options for radius names ("-a", "-r", "--radius", "-b")
    
    * Radius update with changes to address original PR and review comments (#620)
    
    * Update isd_generate.py
    
    add in PROJ like options for radius names ("-a", "-r", "--radius", "-b")
    
    * Update isd_generate.py
    
    updated required statement for --semimajor to catch if only --semiminor or -b is sent. Request that radius is sent in meters and updated some typos.
    
    ---------
    
    Co-authored-by: default avatarAustin Sanders <arsanders@usgs.gov>
    
    ---------
    
    Co-authored-by: default avatarAustin Sanders <arsanders@usgs.gov>
    396524c2
    History
    Name Last commit Last update