-
Notifications
You must be signed in to change notification settings - Fork 579
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
Trilinos Master Merge PR Generator: Auto PR created to promote from master_merge_20200429_071250 branch to master #7282
Merged
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
maps that have some empty processors #7223
Didn't intend to commit this.
This reverts commit e68542f.
The unused code sometimes caused compilation problems (that's why I noticed it)
This allows to check whether the distributor has a reverse distor.
Build several different maps; build DistributedContiguousDirectories from them; getEntries for all Map entries; check for correctness
This bug was affecting only subviews taken in a way that for a LayoutLeft view the first subview argument was a pair of length zero, or for a LayoutRight the last argument was a pair of length zero but all other dimensions are larger than 0. This is a bug which has likely always existed in Kokkos (I checked back to 2.7, but don't see a reason why this wouldn't have been here before).
The new test added as reproducer uses MpiComm directly, which apparently in a serial build fails.
Automatically Merged using Trilinos Pull Request AutoTester PR Title: Stupid changes to resolve FPEs PR Author: bathmatt
Automatically Merged using Trilinos Pull Request AutoTester PR Title: Tpetra Distributor: getReverse changes PR Author: cgcgcg
…previous behavior.
* CONFIG: Update NetCDF and HDF5 TPL version * DO NOT MERGE See if this fixes builds * CONFIG: Update sems netcdf module extension for new versions * NETCDF: Remove BoostLib dependency * NETCDF: See if this triggers use of curl * NETCDF: Better way of specifying library list * HDF5 -- See what HDF5 variables are set to... * Undo some changes that were probably not needed * NETCDF: Try older versions on gcc-4.8.4 build * NETCDF: Try older versions on gcc-4.8.4 build * NETCDF: Remove incorrect boost dependency * Eliminate debug print * Clean up the NetCDF library settings. For SEMS, the pnetcdf library is in the SEMS_NETCDF_ROOT directory and SEM_PNETCDF_ROOT is not defined (or PNETCDF_ROOT). In all scripts checked whether PNETCDF_ROOT was defined and if not, set it to value of NETCDF_ROOT In cmake config files make sure root of libpnetcdf.{a|so} is PNETCDF_ROOT and make sure PNETCDF_ROOT was set to some value. * TPLs updated on test machine Use HDF5-1.10.6 and NetCDF-4.7.3 now. * Eliminate curl library dependency for cuda build The SEMS installations of NetCDF included the DAP setting which requires -lcurl; Installations on other systems did not include DAP, so don't need curl. This is detected in the FindNetCDF.cmake, but that is currently not called for most builds... * Mutrino NetCDF does not require curl library * Ride NetCDF library does not require curl library * shiller NetCDF does not require curl library * van1-tx2 NetCDF library does not require curl library * waterman NetCDF library does not require curl library * Remove debug variable prints * Update the non-unit test file also After sems was updated on test machine, I updated the unit test file on 4.8.4 build, but not the other file that also needed updating... * Curl library dependency not related to MPI The curl library dependency is independent of parallel (MPI) setting. It is a consequence of enabling DAP2 or DAP4 support in NetCDF. It is correctly detected if the FindNetCDF.cmake module is called, but that does not seem to be called for the Trilinos builds that I have done; probably due to the environment variables set before cmake is called?. The sems builds do have DAP enabled and therefore require the curl library. Here we enable the curl dependency if the FindNetCDF.cmake module is not called. This works on all the pull-request builds and several ATDM HPC builds that I have tried. Co-authored-by: Greg Sjaardema <[email protected]>
Automatically Merged using Trilinos Pull Request AutoTester PR Title: Tpetra: unit test to reproduce failure after kokkos 3.1 promotion PR Author: rppawlo
The ATDM APPs are not using this package so no reason to test this as part of ATDM Trilinos testing.
Automatically Merged using Trilinos Pull Request AutoTester PR Title: tpetra: removed unused code and comments that no longer are relevant PR Author: kddevin
…le-compadre Automatically Merged using Trilinos Pull Request AutoTester PR Title: ATDM: Remove new package Compadre from ATDM Trilinos testing (#6934) PR Author: bartlettroscoe
…-cell-normals Automatically Merged using Trilinos Pull Request AutoTester PR Title: Panzer: partial rollback/amendment of 7225 due to EMPIRE reliance on previous behavior. PR Author: CamelliaDPG
* support 32-bit real parmetis / metis libraries * show parmetis version and int/real size in config output * show supported decomp options in config output
CUBLAS and CUSOLVER will become required TPLs, otherwise LAPACK and BLAS will become required TPLs. Fixed tests/examples logic for Compadre package.
…peratorSplitstepper 7188 app action interface for operator splitstepper
…ubcyling-stepper 7236 app action interface for subcyling stepper
- Update boost, cgns, euclid, netcdf, pnetcdf, and sgm tpls for both intel-18 and 19. - Update intel-18 to use openmpi 4.0.1 instead of openmpi 2.0.3.
Automatically Merged using Trilinos Pull Request AutoTester PR Title: Compadre package CMake logic update PR Author: kuberry
…ble-fix Automatically Merged using Trilinos Pull Request AutoTester PR Title: SEACAS: Fix enabling of ParMETIS; clean up PR Author: gsjaardema
Automatically Merged using Trilinos Pull Request AutoTester PR Title: Tpetra: fixed incorrect directory results PR Author: kddevin
Automatically Merged using Trilinos Pull Request AutoTester PR Title: ATDM/common/toss3: Update TPLs PR Author: e10harvey
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_gcc_4.8.4 to start: Total Wait = 603
|
Status Flag 'Master Merge AutoTester' - Failure: Timed out waiting for job Trilinos_pullrequest_intel_17.0.1 to start: Total Wait = 603
|
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_20200429_071250 branch to master