Merge pull request #315 from pbashyal-nmdp/default_redux_is_lgx

Default to `lgx` redux

Author: mmaiers-nmdp

pyard/ard.py

@@ -363,7 +363,7 @@ def _sorted_unique_gl(self, gls: List[str], delim: str) -> str:
         )
 
     @functools.lru_cache(maxsize=DEFAULT_CACHE_SIZE)
-    def redux(self, glstring: str, redux_type: VALID_REDUCTION_TYPES) -> str:
+    def redux(self, glstring: str, redux_type: VALID_REDUCTION_TYPES = "lgx") -> str:
         """
         Does ARD reduction with gl string and reduction type
 

tests/test_pyard.py

@@ -206,8 +206,17 @@ def test_is_null(self):
             self.ard.is_null(allele), msg="MACs shouldn't be called as Nulls"
         )
 
+
+    def test_default_redux_is_lgx(self):
+        allele = "A*24:BKKPV+A*26:03^B*15:BKNTS+B*15:07"
+        lgx_redux = self.ard.redux(allele, "lgx")
+        default_redux = self.ard.redux(allele)
+        self.assertEqual(lgx_redux, default_redux, msg="Default redux should be lgx")
+
+        
     def test_mac_is_reversible(self):
         mac_code = "A*68:AJEBX"
         expanded_mac = self.ard.expand_mac(mac_code)
         lookup_mac = self.ard.lookup_mac(expanded_mac)
         self.assertEqual(mac_code, lookup_mac, msg="MACs should be reversible")
+