Open kledmundson opened 1 month ago
The build and test suite have started for your pull request.
To view your build log, please reference the build with source version: "PR_5600".
Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch.
Two test failures for this PR: 1255 - ThreeImageNetwork.FunctionalTestNoseamDefault (Failed) 1256 - ThreeImageNetwork.FunctionalTestNoseamEvenBoxFilterSamples (Failed)
The build and test suite have started for your pull request.
To view your build log, please reference the build with source version: "PR_5600".
Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch.
@kledmundson Change log needs to be updated here as well
👍 Looks good to me. Just needs merge conflicts fixed before approving.
The build and test suite have started for your pull request.
To view your build log, please reference the build with source version: "PR_5600".
Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch.
@kledmundson Looks like one last merge conflict needs to be resolved here
The build and test suite have started for your pull request.
To view your build log, please reference the build with source version: "PR_5600".
Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch.
Description
The noseam application has been refactored to be callable. The existing Makefile test has been converted to a gtest utilizing the ThreeImageNetwork test fixture. Additional tests have been added. The Makefile test has been removed.
Related Issue
https://github.com/DOI-USGS/ISIS3/issues/5599
How Has This Been Validated?
ctest suite results...
ctest -R Noseam --output-on-failure Test project /Users/kledmundson/ISISDev/noseam/Jul182024/ISIS3/build Start 1255: ThreeImageNetwork.FunctionalTestNoseamDefault 1/5 Test #1255: ThreeImageNetwork.FunctionalTestNoseamDefault ................ Passed 5.65 sec Start 1256: ThreeImageNetwork.FunctionalTestNoseamEvenBoxFilterSamples 2/5 Test #1256: ThreeImageNetwork.FunctionalTestNoseamEvenBoxFilterSamples ... Passed 2.82 sec Start 1257: ThreeImageNetwork.FunctionalTestNoseamEvenBoxFilterLines 3/5 Test #1257: ThreeImageNetwork.FunctionalTestNoseamEvenBoxFilterLines ..... Passed 2.85 sec Start 1258: ThreeImageNetwork.FunctionalTestNoseamNoBoxFilterSamples 4/5 Test #1258: ThreeImageNetwork.FunctionalTestNoseamNoBoxFilterSamples ..... Passed 1.65 sec Start 1259: ThreeImageNetwork.FunctionalTestNoseamNoBoxFilterLines 5/5 Test #1259: ThreeImageNetwork.FunctionalTestNoseamNoBoxFilterLines ....... Passed 1.66 sec
100% tests passed, 0 tests failed out of 5
Total Test time (real) = 14.83 sec
Types of changes
Checklist:
Licensing
This project is mostly composed of free and unencumbered software released into the public domain, and we are unlikely to accept contributions that are not also released into the public domain. Somewhere near the top of each file should have these words: