Unverified Commit eefb6d66 authored by robotprogrammer22's avatar robotprogrammer22 Committed by GitHub
Browse files

Ocam warnings added (#4217) 



* adding warning message to program and documents

* pull request review changes

* updated warning text

Co-authored-by: default avatarJesse Mapel <jam826@nau.edu>
parent aeeca9f4

CHANGELOG.md

+1 −0
Original line number Diff line number Diff line
@@ -32,6 +32,7 @@ update the Unreleased link so that it compares against the latest release tag. 


### Added



- Added warning to ocams2isis about the model being out of date. [#4200](https://github.com/USGS-Astrogeology/ISIS3/issues/4200)

- Added documentation to lronaccal and lrowaccal to describe why there are negative DNs in I/F calibrated images. [#3860](https://github.com/USGS-Astrogeology/ISIS3/issues/3860)

- Update qview MeasureTool to add an option to calculate distances using RA/DEC and update qview to show DEC/RA rather than LAT/LON in lower-right corner [#3371](https://github.com/USGS-Astrogeology/ISIS3/issues/3371)

- Updated spiceinit so that a user can specify a shape model and use the spice web service without any errors. [#1986](https://github.com/USGS-Astrogeology/ISIS3/issues/1986)

isis/src/osirisrex/apps/ocams2isis/main.cpp

+3 −0
Original line number Diff line number Diff line
@@ -23,6 +23,9 @@ void IsisMain() { 


  UserInterface &ui = Application::GetUserInterface();



  QString error_message = "WARNING: This camera model is out of date. See ocam2isis documentation for details.";

  std::cerr << error_message << std::endl;



  ProcessImportFits importFits;



  importFits.setFitsFile(FileName(ui.GetFileName("FROM")));

isis/src/osirisrex/apps/ocams2isis/ocams2isis.xml

+11 −1
Original line number Diff line number Diff line
@@ -10,6 +10,16 @@ 
      This program will import FITS (Flexible Image Transport System) formatted files from the

      OSIRIS-REx MapCam, SamCam, and PolyCam instruments into the ISIS format.

    </p>

    <p>

      The following issues were found by the OSIRIS-REx Image Processing Working Group and have been fixed in their version but are not yet available in the public version:

      <ul>

        <li>Focal position dependent distortion for the PolyCam model is not implemented</li>

        <li>The current distortion is out of date, software changes are needed to use the OpenCV based distortion in the latest IKs</li>

        <li>The Struct SPK is not being used which accounts for the positions of the instruments relative to the center of the spacecraft. From the struct kernel docs this will result in an error of about a meter.</li>

        <li>Only specific sites have DSKs available. Software changes are required to load multiple DSKs at the same time. It is recommended that you only work within the extents of the DSK you are using. Use the dskbrief utility to view the lat/lon extents of your DSK file.</li>

      </ul>

      Based on these issues, it is recommended that users don't attempt to do sub-meter or color processing with this version of the software.

    </p>

  </description>



  <history>