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

slpmap should support negative values #1795

Closed
ascbot opened this issue Feb 13, 2019 · 2 comments
Closed

slpmap should support negative values #1795

ascbot opened this issue Feb 13, 2019 · 2 comments
Labels
bug Something isn't working

Comments

@ascbot
Copy link
Contributor

ascbot commented Feb 13, 2019


Author Name: Trent Hare (Trent Hare)

Original Date: 2013-09-26
Original Assignee: Kimberly Oyama


I don't think there should be anything special about negative values in a DEM or image such that slpmap fails. Hopefully this should be an easy fix. Test files can be any LMMP DEM which crosses 0. Test case:
/archive/projects/lmmp/SOCET_Set/LRO_NAC/LMMP_A15/PRELIMINARY_RELEASE/LRO_NAC_DEM_26N004E_150cmp.cub

slopmap version:
"3.4.4.5390 stable | 2013-06-25"

Based on this discussion site thread:
https://isis.astrogeology.usgs.gov/IsisSupport/index.php/topic,3776.msg15143.html#msg15143

thanks,
Trent

@ascbot ascbot added this to the 3.4.6 (2014-04-04 Apr) milestone Feb 13, 2019
@ascbot ascbot added the bug Something isn't working label Feb 13, 2019
@ascbot
Copy link
Contributor Author

ascbot commented Feb 13, 2019


Original Redmine Comment
Author Name: Kimberly Oyama (Kimberly Oyama)
Original Date: 2014-03-04T20:09:51Z


I've updated the documentation for the AUTOMATIC and USER options. The default value for the resolution when the USER option is selected is the pixel resolution from the input cube. If the input cube doesn't have a projection the user should enter a value for that parameter.

To test, set isis to:

/work/projects/isis/latest/m01764/isis

The documentation can be found at:

file:///work/projects/isis/latest/m01764/isis/src/base/apps/slpmap/slpmap.html

@ascbot
Copy link
Contributor Author

ascbot commented Feb 13, 2019


Original Redmine Comment
Author Name: Trent Hare (Trent Hare)
Original Date: 2014-03-28T01:31:00Z


Seems to be working. Test files I tried were here:
/archive/projects/TOPO/LOLA_FEB2013/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant