Skip to content

Commit

Permalink
Refactor lrowaccal for testability and add a radiance units label to …
Browse files Browse the repository at this point in the history
…the output cube (#4940)

* Add radiance units label in lrowaccal output cube

* Refactor lrowaccal app to be callable for testing and add radiance units to output PVL

* Update formatting and NULL initialization

* Add NULL assignments after deletions

* Add CubeAttributeOutput to output cube initialization

* Add functional test for lrowaccal units label addition and accompanying test cube

* Refactor lrowaccal to remove global variables and convert helper functions to lambdas with captures

* Update history section of lrowaccal XML file with refactoring and output cube units label changes

* Remove old lrowaccal Makefile tests

* Add lrowaccal functional test to ensure that the radiance units label is not inserted for RadiometricType IOF

* Add Cordell Michaud to the .zenodo.json document

* Add lrowaccal changes to CHANGELOG.md

* Clean up lrowaccal includes and leftover comment

* Re-add lrowaccal Makefile tests

* Re-add disclaimer to top of lrowaccal's main.cpp file

* Add input attributes retrieval to lrowaccal.cpp

* Fix lrowaccal functional test error messages and add PVl group and existence checks

* Move lrowaccal units change message to Changed sub-header in CHANGELOG.md

* Simplify lrowaccal functional test assertions

Co-authored-by: Victor Silva <victor.silva@asu.edu>
  • Loading branch information
michaudcordell and victoronline committed May 11, 2022
1 parent aab4563 commit 1facdb7
Show file tree
Hide file tree
Showing 8 changed files with 910 additions and 793 deletions.
4 changes: 4 additions & 0 deletions .zenodo.json
Expand Up @@ -304,6 +304,10 @@
"affiliation": "University of California, Los Angeles",
"name": "Mehlman, Bob"
},
{
"affiliation": "Arizona State University",
"name": "Michaud, Cordell"
},
{
"name": "Milazzo, Moses"
},
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Expand Up @@ -38,6 +38,7 @@ release.
- Added a new application, framestitch, for stitching even and odd push frame images back together prior to processing in other applications. [4924](https://github.com/USGS-Astrogeology/ISIS3/issues/4924)

### Changed
- Updated the LRO calibration application Lrowaccal to add a units label to the RadiometricType keyword of the Radiometry group in the output cube label if the RadiometricType parameter is Radiance. No functionality is changed if the RadiometricType parameter is IOF. Lrowaccal has also been refactored to be callable for testing purposes. Issue: [#4939](https://github.com/USGS-Astrogeology/ISIS3/issues/4939), PR: [#4940](https://github.com/USGS-Astrogeology/ISIS3/pull/4940)

### Added
- Improved functionality of msi2isis and MsiCamera model to support new Eros dataset, including support for Gaskell's SUMSPICE files that adjust timing, pointing and spacecraft position ephemeris. [#4886](https://github.com/USGS-Astrogeology/ISIS3/issues/4886)
Expand Down

0 comments on commit 1facdb7

Please sign in to comment.