Trilinos Master Merge PR Generator: Auto PR created to promote from master_merge_20200516_000615 branch to master #7388
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The OrthoManagerTest had several typos in it, which were not found because all tests that used it were essentially disabled. The OrthoManagerTest has been fixed and all TSQR-specific tests have been placed behind an ifdef. The Epetra interface test for the OrthoManager has been enabled and the dependency on TSQR has been removed due to the change in the tester.
are being run inside cmake (e.g., detect configure vs build time)
Automatically Merged using Trilinos Pull Request AutoTester PR Title: Tacho - export supernodes info PR Author: kyungjoo-kim
Fixes errors in OrthoManagerTest and enables Epetra testing
…or use by Albany (#7359) * Piro: adding a couple member functions to Piro::TransientSolver class. * Piro: cleanups to Piro::TempusSolver. Removing unneeded logic/variables. Renaming member variables in a consistent way. * Piro: more cleanups of Piro::TempusSolver. Adding some unit tests involving transient sensitivities, which will fail for now since I have not written the code for the sensitivities in the Piro::TempusSolver class. * Piro: more cleanups. * Piro: more refactoring/cleanup towards adding transient sensitivities. Moving response stuff to Piro::TransientSolver. Extending code to work with >1 parameters and responses (caveat: this case has not been tested yet). * Piro: adding flags for forward/adjoint sensitivities. Further refactors. * Tempus: replacing DERIV_MV_BY_COL with DERIV_MV_JACOBIAN_FORM and DERIV_TRANS_MV_BY_ROW with DERIV_MV_GRADIENT_FORM. DERIV_MV_BY_COL and DERIV_TRANS_MV_BY_ROW are deprecated in Thyra. * Piro: adding ifdef to allow option of disabling sensivity tests. * Piro: work on transient sensitivities. * Creating Piro::TempusIntegrator class, which is effectively a wrapper class of various integrators from Tempus (IntegratorBasic, IntegratorFowardSensitivity, IntegratorAdjointSensitivity). * Piro: forgot to check this in with previous commit. * Piro: filling in TempusIntegrator routines. * Piro: switching TempusSolver to use new Piro::TempusIntegrator class. This is an interface to the various integrators in Piro, including the sensitivity ones. Working tests pass. * Piro: hooking up setting of sensitivity method based on input parameters for transient sensitivities. * Piro: continuing hooking up of relevant Tempus routines for transient sensitivities. * Piro: filling in response sensitivities code. * Piro: continuing work on transient forward sensitivities. Filling in/cleaning up/documenting response sensitivity calculation. * Piro: adding getDxDp(), getDxdotDp() and getDxdotdotDp() routines to Piro::TempusIntegrator class, as well as Piro::TempusIntegrator member variable to Piro::TransientSolver class. Populating dxdp_mv in evalConvergedModel() using getDxDp(). * Piro: adding getDgDp() routine to Piro::TempusIntegrator - looking forward to adjoint sensitivities. * Piro: correcting comment about clearObservers() routine not existing for sensitivity integrators in Tempus. * Piro: cleaning up verbose output. Removing exception throwing when requesting sensitivities. Refactoring Tempus unit tests into sensitivity ones and those w/o sensitivities. Creating an EpetraExt and Thyra version of the sensitivity tests, for debugging. Sensitivity tests currently fail - need to debug. * Piro: removing LinearOp sensitivity tests for Tempus - forward sensitivities do not work with LinearOp. * Piro: removing unneeded support. * Piro: bug fix affecting tempus solver forward sensitivity unit tests. Now the sensitivity tests all pass on 4 procs but 2 of them fail on 1 proc, which is super weird... Removing forward sensitivity unit tests using the Tpetra version of the MockModel_A, as these were redundant. * Piro: creating SinCos example, analogous to the one in Tempus, for testing forward sensitivities. Test works in serial and parallel - yay! * Piro: some cleanup and renaming of tests for forward sensitivities. * Piro: refactoring transient sensitivity sincos unit tests - creating 4 tests out of one for 'Staggered' vs. 'Combined' sensitivities, and sensitivities w/ and w/o Tangent operator. * Piro: rearranging sensitivity calculation to avoid calling getDxDp for each parameter and response. * Piro: fixing failing sensitivity tests. All tests pass once again, including new sensitivity ones :). * Piro: adding comments and optional print statements checking response sensitivities. * Piro: removing obsolete types DERIV_TRANS_MV_BY_ROW and DERIV_MV_BY_COL for Thyra model evaluator. * Piro: bug fix - sensitivity type was set too late, causing Albany transient tests to fail. * Piro: cleaning up comments. * Piro: adding back missing bracket that got deleted accidentally. * Piro: several improvements as follows. - Removing logic preventing user to start time-integration at time other than 0, which is possible to do now in Tempus. Writing corresponding tests. - Extending Piro::ObserverToTempusIntegrationObserverAdapter class so that it handles correctly sensitivities. Adding a test for sensitivities + observer. * Piro: removing __PRETTY_FUNCTION__ debug output. Fixing a couple of the errors caught by the auto-tester. * Piro: removing debug output from tests. * Piro: removing initialConditionModel argument/member variable from Tempus classes. It is not needed/used and a remnant from the Rythmos solver. * Piro: corrections to createOutArgs, which was not implemented earlier for >1 parameter and response. * Piro: changing final difference check to relative difference. * Piro: changing int sens_method to enum. * Piro: removing unused variable. * Piro: fixing Thyra type mismatch that was causing some of the auto-tester tests to fail. * Piro: renaming Piro_TempusHelpers.hpp file and removing unneeded enums * Piro: correcting include. Co-authored-by: Irina Tezaur <[email protected]>
Automatically Merged using Trilinos Pull Request AutoTester PR Title: Provide per-file build statistics for Trilinos and others... PR Author: jjellio
|
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request. |
|
Status Flag 'Master Merge AutoTester' - Failure: Timed out waiting for job Trilinos_pullrequest_cuda_9.2 to start: Total Wait = 603
|
|
Status Flag 'Master Merge AutoTester' - Failure: Timed out waiting for job Trilinos_pullrequest_cuda_9.2 to start: Total Wait = 603
|
jennloe
added
the
AT: RETEST
|
Status Flag 'Master Merge AutoTester' - User Requested Retest - Label AT: RETEST will be reset after testing. |
|
Status Flag 'Master Merge AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Build InformationTest Name: Trilinos_pullrequest_gcc_4.8.4
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_intel_17.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_4.9.3_SERIAL
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_7.2.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_8.3.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_7.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_9.0.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_cuda_9.2
Jenkins Parameters
Using Repos:
Pull Request Author: trilinos-autotester |
|
Status Flag 'Master Merge AutoTester' - Jenkins Testing: all Jobs PASSED Pull Request Auto Testing has PASSED (click to expand)Build InformationTest Name: Trilinos_pullrequest_gcc_4.8.4
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_intel_17.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_4.9.3_SERIAL
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_7.2.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_8.3.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_7.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_9.0.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_cuda_9.2
Jenkins Parameters
|
trilinos-autotester
removed
the
AT: RETEST
|
Status Flag 'Pre-Merge Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request. |
|
Status Flag 'Master Merge AutoTester' - Pull Request will be Automerged |
|
Merge on Pull Request# 7388: IS A SUCCESS - Pull Request successfully merged |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Auto PR created to promote from master_merge_20200516_000615 branch to master