Skip to content
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

Initial adjustment of Primary Tested set of Trilinos Packages and TPLs #410

Closed
bartlettroscoe opened this issue Jun 1, 2016 · 20 comments
Closed
Assignees
Labels
Framework tasks Framework tasks (used internally by Framework team) story The issue corresponds to a Kanban Story (vs. Epic or Task)

Comments

@bartlettroscoe
Copy link
Member

bartlettroscoe commented Jun 1, 2016

Next Action Status:

Sent email to SNL customers of Trilinos, asking for list of Trilinos packages and TPLs. Next: Wait for responses ...

CC: @jwillenbring, @bmpersc, @maherou, @trilinos/framework

Blocked By: #158

Blocking: #370

Description:

This story is to do an initial adjustment (mostly an extension) the set of Primary Tested (PT) TPLs to include HDF5, Netcdf, Boost, SWIG and any others needed to fully build and test packages like SEACAS, STK, etc. This initial set of packages and TPLs will be limited to what is provided by the SEMS Dev Env (see #158). Otherwise, some packages that could be included based on that set of TPLs should otherwise be excluded from the set of PT packages as guided by the needs of important Trilinos customers and critical research efforts (which are need to be well protected). Purely exploratory software or packages without important current users uses should generally be excluded from the PT set (to speed up the builds and tests). Also, non-critical configurations (e.g. complex, float) should be avoided to speed up builds and testing.

This initial story will be to support the usage of the checkin-test.py script to update from the 'develop' to the 'master' branch (see #370).

Tasks:

  1. Get initial feedback from Trilinos developers and stakeholders (captured in emails to Ross) [Done]
  2. Document, analyze and combine the lists from SNL customers (see TRIL-59) ...
  3. Come up with a new list of PT Packages and TPLs (that can be supported by the SEMS Dev Env) ...
  4. Fill out the "Testing Category" column in the Trilinos Package Classification Database ...
  5. Update the file Trilinos/PackagesList.cmake and TPLsList.cmake
@bartlettroscoe bartlettroscoe added Framework tasks Framework tasks (used internally by Framework team) stage: ready The issue is ready to be worked in a Kanban-like process labels Jun 1, 2016
@bartlettroscoe bartlettroscoe added the story The issue corresponds to a Kanban Story (vs. Epic or Task) label Jun 1, 2016
@bartlettroscoe
Copy link
Member Author

bartlettroscoe commented Jun 26, 2016

I just sent an email to internal SNL customers of Trilinos in order to try to gather up the list of important Trilinos packages. Unfortunately, I can't include those emails here because they name specific SNL codes. But I will gather up the final list in this Issue ticket.

@bartlettroscoe bartlettroscoe added stage: in progress Work on the issue has started and removed stage: ready The issue is ready to be worked in a Kanban-like process labels Jun 26, 2016
@bartlettroscoe
Copy link
Member Author

bartlettroscoe commented Jul 20, 2016

NOTE: I just created the internal SON ticket:

to process the list of "important" packages and TPLs for internal SNL customers. The outcome from that story will be an updated candidate list of PT packages and TPLs.

@bartlettroscoe
Copy link
Member Author

bartlettroscoe commented Nov 9, 2016

I completed the analysis of the Trilinos packages, SE package, and TPLs used by a bunch of Trilinos customers and created a combined list from all of these. You can see what customers were considered and where the data came from by looking at:

The combined list sorted alphabetically is given below. The challenge that we face is that large set of TPLs that all of our Trilinos customers are using:

  • AMD BLACS BLAS Boost BoostLib DLlib HDF5 LAPACK METIS MPI MUMPS Netcdf PARDISO_MKL ParMETIS PETSC Pthread QT SCALAPACK SuperLU SuperLUDist UMFPACK X11 y12m Zlib

The problem is that the SEMS env can only provide the TPLs:

  • Pthread MPI BLAS LAPACK Boost ParMETIS Zlib HDF5 Netcdf SuperLU BoostLib DLlib

What is missing from SEMS is:

  • AMD, BLASC, MUMPS, PARDISO_MKL, PETSC, QT, SCALAPACK, SuperLUDist, UMFPACK, y12m

Not much we can do about that if we are going to build our CI system on the SEMS env. It has what it has.

I am going to put this story into review now and move on with #482.

@trilinos/framework, any comments on this?


Full set of packages:

Amesos Amesos2 Anasazi AztecOO Belos Claps Domi Drekar DrekarMHD Epetra EpetraExt FEI Galeri GlobiPack Gtest Ifpack Ifpack2 Intrepid Intrepid2 Isorropia Kokkos Mesquite ML MOOCHO MueLu NOX OptiPack Pamgen Panzer Phalanx Pike Piro ROL RTOp Rythmos Sacado SEACAS Shards ShyLU STK Stokhos Stratimikos Teko Teuchos ThreadPool Thyra Tpetra TriKota TrilinosCouplings Triutils Xpetra Zoltan Zoltan2

Full set of SE packages:

Amesos Amesos2 Anasazi AztecOO Belos Claps Domi Drekar DrekarMHD Epetra EpetraExt FEI Galeri GlobiPack Gtest Ifpack Ifpack2 Intrepid Intrepid2 IntrepidCore Isorropia Kokkos KokkosAlgorithms KokkosContainers KokkosCore KokkosTPL Mesquite ML MOOCHO MueLu NOX OptiPack Pamgen Panzer PanzerAdaptersSTK PanzerCore PanzerDiscFE PanzerDofMgr Phalanx Pike PikeBlackBox PikeImplicit Piro ROL RTOp Rythmos Sacado SEACAS SEACASAlgebra SEACASAprepro SEACASAprepro_lib SEACASBlot SEACASChaco SEACASConjoin SEACASEjoin SEACASEpu SEACASEx1ex2v2 SEACASEx2ex1v2 SEACASExo_format SEACASExodiff SEACASExodus SEACASExodus_for SEACASExoIIv2for32 SEACASExomatlab SEACASExotxt SEACASFastq SEACASGen3D SEACASGenshell SEACASGjoin SEACASGrepos SEACASGrope SEACASIoss SEACASMapvar SEACASMapvar-kd SEACASMapvarlib SEACASNemesis SEACASNemslice SEACASNemspread SEACASNumbers SEACASPLT SEACASSupes SEACASSuplib SEACASSuplibC SEACASSuplibCpp SEACASSVDI SEACASTxtexo Shards ShyLU ShyLUCore STK STKClassic STKDoc_tests STKExprEval STKIO STKMesh STKSearch STKSearchUtil STKTopology STKTransfer STKUnit_test_utils STKUnit_tests STKUtil Stokhos Stratimikos Teko Teuchos TeuchosComm TeuchosCore TeuchosKokkosComm TeuchosKokkosCompat TeuchosNumerics TeuchosParameterList TeuchosRemainder ThreadPool Thyra ThyraCore ThyraEpetraAdapters ThyraEpetraExtAdapters ThyraTpetraAdapters Tpetra TpetraClassic TpetraCore TpetraKernels TpetraTSQR TriKota TrilinosCouplings Triutils Xpetra Zoltan Zoltan2

Full set of TPLs:

AMD BLACS BLAS Boost BoostLib DLlib HDF5 LAPACK METIS MPI MUMPS Netcdf PARDISO_MKL ParMETIS PETSC Pthread QT SCALAPACK SuperLU SuperLUDist UMFPACK X11 y12m Zlib

@bartlettroscoe bartlettroscoe added stage: in review Primary work is completed and now is just waiting for human review and/or test feedback and removed stage: in progress Work on the issue has started labels Nov 9, 2016
@srajama1
Copy link
Contributor

srajama1 commented Nov 9, 2016

I see Scotch is missing.

@bartlettroscoe
Copy link
Member Author

bartlettroscoe commented Nov 9, 2016

I see Scotch is missing.

That is interesting. Looks like none of the internal Trilinos customers that responded to my request are using Scotch with Trilinos. You can see the full set of customers that were considered in:

But also note that the SEMS env only provides a 32-bit build for Scotch which is in-compatible with the 64-bit version they provide for ParMETIS. You can read about that in #475.

@srajama1
Copy link
Contributor

srajama1 commented Nov 9, 2016

Scotch is a recent addition. SIERRA officially started testing it late September, if my memory is right.

@bartlettroscoe
Copy link
Member Author

Scotch is a recent addition. SIERRA officially started testing it late September, if my memory is right.

Good to know, thanks. I did a search of the SEMS JIRA project and I found:

that suggests that SEMS might now provide a compatible 64-bit version of Scotch. Can anyone confirm that? @kddevin, have you been able to test out the SEMS env with 64-bit ParMETIS and Scotch both enabled with Zoltan and Zoltan2?

@srajama1
Copy link
Contributor

srajama1 commented Nov 9, 2016

Is there a 32-bit version. Tacho works with 32-bit. We can add a 64-bit version interface as well.
@kyungjoo-kim

@bartlettroscoe
Copy link
Member Author

Is there a 32-bit version. Tacho works with 32-bit. We can add a 64-bit version interface as well.

The problem is that you can't enabled both the 64-bit version of ParMETIS and the 32-bit version of Scotch at the same time. If you do, a bunch of Zoltan and Zoltan2 tests fail. That is no good.

@kyungjoo-kim
Copy link
Contributor

Tacho does not require for 32 bit version specifically. Does SIERRA use a 32 bit version ?

@kddevin
Copy link
Contributor

kddevin commented Nov 9, 2016

  1. Zoltan2 works fine with mixed 32-bit/64-bit TPL index types. Only Zoltan requires consistent ID sizes among the TPLs; it is unlikely we will ever have resources to remove that constraint.
  2. All Zoltan and Zoltan2 tests pass with SEMS' 64-bit ParMETIS and Scotch, and with SEMS' 32-bit ParMETIS and Scotch.

@bartlettroscoe
Copy link
Member Author

Zoltan2 works fine with mixed 32-bit/64-bit TPL index types. Only Zoltan requires consistent ID sizes among the TPLs; it is unlikely we will ever have resources to remove that constraint.

All Zoltan and Zoltan2 tests pass with SEMS' 64-bit ParMETIS and Scotch, and with SEMS' 32-bit ParMETIS and Scotch.

@kddevin, has anyone tried this with the ParMETIS and Scotch TPLs provided by the SEMS Env? If not, we can try this later once we get #482 done.

Adding more TPLs to our CI and Nightly testing is something that we will need to do gradually as we can. (We just need infrastructure manpower to help with this). If you look at the list of TPLs that are actually used by our important Sandia Trilinos customers listed above, you will see that we are falling far short with what Trilinos is currently testing in Nightly builds and what is provided by the SEMS env.

@kddevin
Copy link
Contributor

kddevin commented Nov 9, 2016

@bartlettroscoe I don't understand. What is "this" in your question above?

@bartlettroscoe
Copy link
Member Author

What is "this" in your question above?

Sorry, I should have been more explicit. Has any Zoltan or Zoltan2 developer tried the updated SEMS-provided 64-bit Scotch and ParMETIS TPLs? It looks like these are now present under:

  • /projects/sems/install/rhel6-x86_64/sems/tpl/scotch/6.0.3/gcc/4.8.4/openmpi/1.8.7/nopthread_64bit_parallel/
  • /projects/sems/install/rhel6-x86_64/sems/tpl/parmetis/4.0.3/gcc/4.8.4/openmpi/1.8.7/64bit_parallel/

If not, that is fine. I was just asking.

@kddevin
Copy link
Contributor

kddevin commented Nov 9, 2016

Yes; see my comment 2 above. All tested. All good. Thanks.

@bartlettroscoe
Copy link
Member Author

Realized that we need to make a few changes to the list of packages and SE packages listed above.

First, Claps is an EX. It is not actively developed in Trilinos and there is no reason to test it.

Second, I accidentally listed Drekar and DrekarMHD. Those are not even Trilinos packages.

Third, TriKota depends on Dakota. I don't think we should be forcing Trilinos developers to have to build Dakota when the test Trilinos. I think that is asking too much. Also, there is not a solid integration model for what version of Dakota is compatible with TriKota. Therefore, I think it makes sense to just let Albany test TriKota.

Forth, Mesquite and MOOCHO are in a separate git repo from the main Trilinos repo. Therefore, it makes no sense to include it in the set of PT packages.

So taking out Claps, Drekar, DrekarMHD, Mesquite, and TriKota, the new list of primary tested packages and TPLs becomes:

  • New set of PT packages: Amesos Amesos2 Anasazi AztecOO Belos Domi Epetra EpetraExt FEI Galeri GlobiPack Gtest Ifpack Ifpack2 Intrepid Intrepid2 Isorropia Kokkos ML MueLu NOX OptiPack Pamgen Panzer Phalanx Pike Piro ROL RTOp Rythmos Sacado SEACAS Shards ShyLU STK Stokhos Stratimikos Teko Teuchos ThreadPool Thyra Tpetra TrilinosCouplings Triutils Xpetra Zoltan Zoltan2
  • New set of PT SE packages: Amesos Amesos2 Anasazi AztecOO Belos Domi Epetra EpetraExt FEI Galeri GlobiPack Gtest Ifpack Ifpack2 Intrepid Intrepid2 IntrepidCore Isorropia Kokkos KokkosAlgorithms KokkosContainers KokkosCore KokkosTPL ML MueLu NOX OptiPack Pamgen Panzer PanzerAdaptersSTK PanzerCore PanzerDiscFE PanzerDofMgr Phalanx Pike PikeBlackBox PikeImplicit Piro ROL RTOp Rythmos Sacado SEACAS SEACASAlgebra SEACASAprepro SEACASAprepro_lib SEACASBlot SEACASChaco SEACASConjoin SEACASEjoin SEACASEpu SEACASEx1ex2v2 SEACASEx2ex1v2 SEACASExo_format SEACASExodiff SEACASExodus SEACASExodus_for SEACASExoIIv2for32 SEACASExomatlab SEACASExotxt SEACASFastq SEACASGen3D SEACASGenshell SEACASGjoin SEACASGrepos SEACASGrope SEACASIoss SEACASMapvar SEACASMapvar-kd SEACASMapvarlib SEACASNemesis SEACASNemslice SEACASNemspread SEACASNumbers SEACASPLT SEACASSupes SEACASSuplib SEACASSuplibC SEACASSuplibCpp SEACASSVDI SEACASTxtexo Shards ShyLU ShyLUCore STK STKClassic STKDoc_tests STKExprEval STKIO STKMesh STKSearch STKSearchUtil STKTopology STKTransfer STKUnit_test_utils STKUnit_tests STKUtil Stokhos Stratimikos Teko Teuchos TeuchosComm TeuchosCore TeuchosKokkosComm TeuchosKokkosCompat TeuchosNumerics TeuchosParameterList TeuchosRemainder ThreadPool Thyra ThyraCore ThyraEpetraAdapters ThyraEpetraExtAdapters ThyraTpetraAdapters Tpetra TpetraClassic TpetraCore TpetraKernels TpetraTSQR TriKota TrilinosCouplings Triutils Xpetra Zoltan Zoltan2
  • New set of PT TPLs: Pthread MPI BLAS LAPACK Boost ParMETIS Zlib HDF5 Netcdf SuperLU BoostLib DLlib

bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Nov 10, 2016
I changed this while looking to make sure the Tpetra packages were declared
correctly.
bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Nov 10, 2016
Several packages were added to PT and a few were removed according what is
being used by important Trilinos customers (see trilinos#410 and
trilinos#482).

Also, several TPLs were added to the set of PT TPLs.  These are the TPLs that
are used by important Trilinos customers (see trilinos#410) and are
also provided by the SEMS env.
@bartlettroscoe
Copy link
Member Author

Another adjustment needs to be made. The subpackage IntrepidCore no longer exists in Trilinos. It was taken out in the commit:

commit 889d5e8e493cafe3ef151d6b886e815053e704a5
Author: avdelor <avdelor@sandia.gov>
Date:   Tue Jan 5 16:50:14 2016 -0700

    Intrepid:

    fixed phalanx dependencies
    took intrepid out of core subpackage

diff --git a/packages/intrepid/cmake/Dependencies.cmake b/packages/intrepid/cmake/Dependencies.cmake
index f1b64a3..c2eba4a 100644
--- a/packages/intrepid/cmake/Dependencies.cmake
+++ b/packages/intrepid/cmake/Dependencies.cmake
@@ -1,8 +1,3 @@
-SET(SUBPACKAGES_DIRS_CLASSIFICATIONS_OPTREQS
-  #SubPackageName       Directory         Class    Req/Opt
-  #
-  Core                  core               PT      OPTIONAL
-  )
 SET(LIB_REQUIRED_DEP_PACKAGES Teuchos Shards Sacado)
 SET(LIB_OPTIONAL_DEP_PACKAGES KokkosCore KokkosAlgorithms)
 SET(TEST_REQUIRED_DEP_PACKAGES)

CASL VERA is using very old version of Intrepid before this commit was made.

Removing IntrepidCore the new set of of PT package are:

  • New set of PT packages: Amesos Amesos2 Anasazi AztecOO Belos Domi Epetra EpetraExt FEI Galeri GlobiPack Gtest Ifpack Ifpack2 Intrepid Intrepid2 Isorropia Kokkos ML MueLu NOX OptiPack Pamgen Panzer Phalanx Pike Piro ROL RTOp Rythmos Sacado SEACAS Shards ShyLU STK Stokhos Stratimikos Teko Teuchos ThreadPool Thyra Tpetra TrilinosCouplings Triutils Xpetra Zoltan Zoltan2
  • New set of PT SE packages: Amesos Amesos2 Anasazi AztecOO Belos Domi Epetra EpetraExt FEI Galeri GlobiPack Gtest Ifpack Ifpack2 Intrepid Intrepid2 Isorropia Kokkos KokkosAlgorithms KokkosContainers KokkosCore KokkosTPL ML MueLu NOX OptiPack Pamgen Panzer PanzerAdaptersSTK PanzerCore PanzerDiscFE PanzerDofMgr Phalanx Pike PikeBlackBox PikeImplicit Piro ROL RTOp Rythmos Sacado SEACAS SEACASAlgebra SEACASAprepro SEACASAprepro_lib SEACASBlot SEACASChaco SEACASConjoin SEACASEjoin SEACASEpu SEACASEx1ex2v2 SEACASEx2ex1v2 SEACASExo_format SEACASExodiff SEACASExodus SEACASExodus_for SEACASExoIIv2for32 SEACASExomatlab SEACASExotxt SEACASFastq SEACASGen3D SEACASGenshell SEACASGjoin SEACASGrepos SEACASGrope SEACASIoss SEACASMapvar SEACASMapvar-kd SEACASMapvarlib SEACASNemesis SEACASNemslice SEACASNemspread SEACASNumbers SEACASPLT SEACASSupes SEACASSuplib SEACASSuplibC SEACASSuplibCpp SEACASSVDI SEACASTxtexo Shards ShyLU ShyLUCore STK STKClassic STKDoc_tests STKExprEval STKIO STKMesh STKSearch STKSearchUtil STKTopology STKTransfer STKUnit_test_utils STKUnit_tests STKUtil Stokhos Stratimikos Teko Teuchos TeuchosComm TeuchosCore TeuchosKokkosComm TeuchosKokkosCompat TeuchosNumerics TeuchosParameterList TeuchosRemainder ThreadPool Thyra ThyraCore ThyraEpetraAdapters ThyraEpetraExtAdapters ThyraTpetraAdapters Tpetra TpetraClassic TpetraCore TpetraKernels TpetraTSQR TriKota TrilinosCouplings Triutils Xpetra Zoltan Zoltan2
  • New set of PT TPLs: Pthread MPI BLAS LAPACK Boost ParMETIS Zlib HDF5 Netcdf SuperLU BoostLib DLlib

@bartlettroscoe
Copy link
Member Author

bartlettroscoe commented Nov 10, 2016

Another change to the list of SE packages. Seems that Kokkos no longer has a subpackage called "KokkosTPL". It was removed in this commit:

commit e1e44072a7e8c1c76037f4c7766f321062b2dd6c
Author: Carter Edwards <hcedwar@sandia.gov>
Date:   Tue Apr 7 10:45:15 2015 -0600

    Kokkos: Remove unused TPL subdirectory & subpackage.

D       packages/kokkos/TPL/CMakeLists.txt
M       packages/kokkos/config/configure_tpetra_kokkos_cuda_nvcc_wrapper.sh
M       packages/kokkos/core/cmake/Dependencies.cmake

Only one Trilinos customer listed that (you can see which one by examining the combined_list_of_trlinos_customer_enables.sh script in TRIL-59). That customer must have been using a very old version of Trilinos when they reported this.

The update list of enables removing KokkosTPL is:

  • New set of PT packages: Amesos Amesos2 Anasazi AztecOO Belos Domi Epetra EpetraExt FEI Galeri GlobiPack Gtest Ifpack Ifpack2 Intrepid Intrepid2 Isorropia Kokkos ML MueLu NOX OptiPack Pamgen Panzer Phalanx Pike Piro ROL RTOp Rythmos Sacado SEACAS Shards ShyLU STK Stokhos Stratimikos Teko Teuchos ThreadPool Thyra Tpetra TrilinosCouplings Triutils Xpetra Zoltan Zoltan2
  • New set of PT SE packages: Amesos Amesos2 Anasazi AztecOO Belos Domi Epetra EpetraExt FEI Galeri GlobiPack Gtest Ifpack Ifpack2 Intrepid Intrepid2 Isorropia Kokkos KokkosAlgorithms KokkosContainers KokkosCore ML MueLu NOX OptiPack Pamgen Panzer PanzerAdaptersSTK PanzerCore PanzerDiscFE PanzerDofMgr Phalanx Pike PikeBlackBox PikeImplicit Piro ROL RTOp Rythmos Sacado SEACAS SEACASAlgebra SEACASAprepro SEACASAprepro_lib SEACASBlot SEACASChaco SEACASConjoin SEACASEjoin SEACASEpu SEACASEx1ex2v2 SEACASEx2ex1v2 SEACASExo_format SEACASExodiff SEACASExodus SEACASExodus_for SEACASExoIIv2for32 SEACASExomatlab SEACASExotxt SEACASFastq SEACASGen3D SEACASGenshell SEACASGjoin SEACASGrepos SEACASGrope SEACASIoss SEACASMapvar SEACASMapvar-kd SEACASMapvarlib SEACASNemesis SEACASNemslice SEACASNemspread SEACASNumbers SEACASPLT SEACASSupes SEACASSuplib SEACASSuplibC SEACASSuplibCpp SEACASSVDI SEACASTxtexo Shards ShyLU ShyLUCore STK STKClassic STKDoc_tests STKExprEval STKIO STKMesh STKSearch STKSearchUtil STKTopology STKTransfer STKUnit_test_utils STKUnit_tests STKUtil Stokhos Stratimikos Teko Teuchos TeuchosComm TeuchosCore TeuchosKokkosComm TeuchosKokkosCompat TeuchosNumerics TeuchosParameterList TeuchosRemainder ThreadPool Thyra ThyraCore ThyraEpetraAdapters ThyraEpetraExtAdapters ThyraTpetraAdapters Tpetra TpetraClassic TpetraCore TpetraKernels TpetraTSQR TriKota TrilinosCouplings Triutils Xpetra Zoltan Zoltan2
  • New set of PT TPLs: Pthread MPI BLAS LAPACK Boost ParMETIS Zlib HDF5 Netcdf SuperLU BoostLib DLlib

@bartlettroscoe
Copy link
Member Author

I went back and put the script combined_list_of_trlinos_customer_enables.sh under version control and carefully make all of the tweaks to it that are described above and few others (see TRIL-59). Now I can get a local configure of Trilinos to exactly match the updated list and it looks right.

Here is the updated list of PT enables:

  • New set of PT packages: Amesos Amesos2 Anasazi AztecOO Belos Domi Epetra EpetraExt FEI Galeri GlobiPack Gtest Ifpack Ifpack2 Intrepid Intrepid2 Isorropia Kokkos ML MueLu NOX OptiPack Pamgen Panzer Phalanx Pike Piro ROL RTOp Rythmos Sacado SEACAS Shards ShyLU STK Stokhos Stratimikos Teko Teuchos ThreadPool Thyra Tpetra TrilinosCouplings Triutils Xpetra Zoltan Zoltan2
  • New set of PT SE packages: Amesos Amesos2 Anasazi AztecOO Belos Domi Epetra EpetraExt FEI Galeri GlobiPack Gtest Ifpack Ifpack2 Intrepid Intrepid2 Isorropia Kokkos KokkosAlgorithms KokkosContainers KokkosCore ML MueLu NOX OptiPack Pamgen Panzer PanzerAdaptersSTK PanzerCore PanzerDiscFE PanzerDofMgr Phalanx Pike PikeBlackBox PikeImplicit Piro ROL RTOp Rythmos Sacado SEACAS SEACASAlgebra SEACASAprepro SEACASAprepro_lib SEACASChaco SEACASConjoin SEACASEjoin SEACASEpu SEACASEx1ex2v2 SEACASEx2ex1v2 SEACASExo_format SEACASExodiff SEACASExodus SEACASExodus_for SEACASExoIIv2for32 SEACASExomatlab SEACASExotxt SEACASGen3D SEACASGenshell SEACASGjoin SEACASGrepos SEACASGrope SEACASIoss SEACASMapvar SEACASMapvar-kd SEACASMapvarlib SEACASNemesis SEACASNemslice SEACASNemspread SEACASNumbers SEACASSupes SEACASSuplib SEACASSuplibC SEACASSuplibCpp SEACASTxtexo Shards ShyLU ShyLUCore STK STKDoc_tests STKExprEval STKIO STKMesh STKSearch STKSearchUtil STKTopology STKTransfer STKUnit_test_utils STKUnit_tests STKUtil Stokhos Stratimikos Teko Teuchos TeuchosComm TeuchosCore TeuchosKokkosComm TeuchosKokkosCompat TeuchosNumerics TeuchosParameterList TeuchosRemainder ThreadPool Thyra ThyraCore ThyraEpetraAdapters ThyraEpetraExtAdapters ThyraTpetraAdapters Tpetra TpetraClassic TpetraCore TpetraKernels TpetraTSQR TrilinosCouplings Triutils Xpetra Zoltan Zoltan2
  • New set of PT TPLs: Pthread MPI BLAS LAPACK Boost ParMETIS Zlib HDF5 Netcdf SuperLU BoostLib DLlib

bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Nov 11, 2016
Since Panzer has been elivated to PT and since important Trilinos customers
are using these subpackages, they must be elevated to PT.  See
trilinos#410 and trilinos#482 for more details.
@bartlettroscoe bartlettroscoe self-assigned this Nov 12, 2016
bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Nov 28, 2016
I changed this while looking to make sure the Tpetra packages were declared
correctly.
bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Nov 28, 2016
Several packages were added to PT and a few were removed according what is
being used by important Trilinos customers (see trilinos#410 and
trilinos#482).

Also, several TPLs were added to the set of PT TPLs.  These are the TPLs that
are used by important Trilinos customers (see trilinos#410) and are
also provided by the SEMS env.
bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Nov 28, 2016
This is being used by an important Trilinos customer (see trilinos#410 and TRIL-59)
and therefore should be tested in pre-push CI testing.
bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Nov 28, 2016
Since Pike has been elevated to PT so must its subpackages that are used by an
important Trilinos customer (CASL VERA in this case).
bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Nov 28, 2016
Since SEACAS has been elevated to PT and since important Trilinos customers
are using these subpackages, they must be elevated to PT.  See
trilinos#410 and trilinos#482 for more details.
bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Nov 28, 2016
Since STK has been elivated to PT and since important Trilinos customers are
using these subpackages, they must be elevated to PT.  See
trilinos#410 and trilinos#482 for more details.
bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Nov 28, 2016
Since Panzer has been elivated to PT and since important Trilinos customers
are using these subpackages, they must be elevated to PT.  See
trilinos#410 and trilinos#482 for more details.
bartlettroscoe added a commit to bartlettroscoe/seacas that referenced this issue Nov 29, 2016
…/Trilinos#410)

These are the subpackage of SEACAS that are used by Trilinos customers and
therefore have been elevated to Primary Test (PT) for Trilinos (see
trilinos/Trilinos#410).

I also replaced the deprecated test category 'SS' (Secondary Stable) with 'ST'
(Secondary Tested).

This is a duplicate of the direct Trilinos SEACAS commit:

    8057316 "SEACAS: Elevate many subpackages to PT (sandialabs#410, sandialabs#482)"
    Author: Roscoe A. Bartlett <rabartl@sandia.gov>
    Date:   Thu Nov 10 08:05:06 2016 -0700 (24 hours ago)

    M       packages/seacas/cmake/Dependencies.cmake

Therefore, that commit will need to be reverted (using 'git revert 8057316')
before snapshotting an updated version of SEACAS into Trilinos.

See the Trilinos issues:
* trilinos/Trilinos#410
* trilinos/Trilinos#482
bartlettroscoe added a commit to bartlettroscoe/seacas that referenced this issue Nov 29, 2016
…/Trilinos#410)

These are the subpackage of SEACAS that are used by Trilinos customers and
therefore have been elevated to Primary Test (PT) for Trilinos (see
trilinos/Trilinos#410).  I manually verified that this gives the right set of
PT packages by configuring Trilinos with this version of the SEACAS package.
(That is, I cloned the seacas repo under Trilinos then configured Trilinos
with -DSEACAS_SOURCE_DIR_OVERRIDE:STRING=seacas/packages/seacas).

I also replaced the deprecated test category 'SS' (Secondary Stable) with 'ST'
(Secondary Tested).

This is a duplicate of the direct Trilinos SEACAS commit:

    8057316 "SEACAS: Elevate many subpackages to PT (sandialabs#410, sandialabs#482)"
    Author: Roscoe A. Bartlett <rabartl@sandia.gov>
    Date:   Thu Nov 10 08:05:06 2016 -0700 (24 hours ago)

    M       packages/seacas/cmake/Dependencies.cmake

Therefore, that commit will need to be reverted (using 'git revert 8057316')
before snapshotting an updated version of SEACAS into Trilinos (or updating
Trilinos in that other application code that snapshots Trilinos+SEACAS that
also owns the STK source that I can't mention).

See the Trilinos issues:
* trilinos/Trilinos#410
* trilinos/Trilinos#482
gsjaardema added a commit to sandialabs/seacas that referenced this issue Nov 30, 2016
CONFIG: Elevate many subpackages to PT for Trilinos testing (trilinos/Trilinos#410)
@bartlettroscoe
Copy link
Member Author

This is complete and is being used in new CI build. It is unlikely that anyone will seriously review this so closing as complete.

Also, we are no longer going to purse extending the checkin-test-sems.sh (or checkin-test.py) script for Trilinos (see #482 (comment)). No idea what selection of packages and TPLs will be used for new automatic PR system but that can be determined later by someone else.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework tasks Framework tasks (used internally by Framework team) story The issue corresponds to a Kanban Story (vs. Epic or Task)
Projects
None yet
Development

No branches or pull requests

4 participants