Updated and cleaned up the recipe for the LBANN and Hydrogen packages. (#14612)

* Updated and cleaned up the recipe for the LBANN and Hydrogen packages.

* Flake8

* Added type test for catch2

var/spack/repos/builtin/packages/hydrogen/package.py

@@ -19,6 +19,7 @@ class Hydrogen(CMakePackage):
     maintainers = ['bvanessen']
 
     version('develop', branch='hydrogen')
+    version('1.3.3', sha256='140112066b84d33ca4b75c8e520fb15748fa648c4d2b934c1eb5510173ede5f5')
     version('1.3.2', sha256='50bc5e87955f8130003d04dfd9dcad63107e92b82704f8107baf95b0ccf98ed6')
     version('1.3.1', sha256='a8b8521458e9e747f2b24af87c4c2749a06e500019c383e0cefb33e5df6aaa1d')
     version('1.3.0', sha256='0f3006aa1d8235ecdd621e7344c99f56651c6836c2e1bc0cf006331b70126b36')
@@ -58,6 +59,8 @@ class Hydrogen(CMakePackage):
             description='Builds with Aluminum communication library')
     variant('omp_taskloops', default=False,
             description='Use OpenMP taskloops instead of parallel for loops.')
+    variant('half', default=True,
+            description='Builds with support for FP16 precision data types')
 
     # Note that #1712 forces us to enumerate the different blas variants
     depends_on('openblas', when='blas=openblas ~openmp_blas ~int64_blas')
@@ -94,6 +97,7 @@ class Hydrogen(CMakePackage):
 
     depends_on('cuda', when='+cuda')
     depends_on('cub', when='+cuda')
+    depends_on('half', when='+half')
 
     conflicts('@0:0.98', msg="Hydrogen did not exist before v0.99. " +
               "Did you mean to use Elemental instead?")
@@ -127,6 +131,7 @@ def cmake_args(self):
             '-DHydrogen_ENABLE_CUB=%s' % ('+cuda' in spec),
             '-DHydrogen_ENABLE_CUDA=%s' % ('+cuda' in spec),
             '-DHydrogen_ENABLE_TESTING=%s' % ('+test' in spec),
+            '-DHydrogen_ENABLE_HALF=%s' % ('+half' in spec),
         ]
 
         # Add support for OS X to find OpenMP