From 5e882752893cce31f9b510609b8096c1ec32aabc Mon Sep 17 00:00:00 2001
From: Christoph Groth <christoph.groth@cea.fr>
Date: Tue, 10 Nov 2015 12:12:18 +0100
Subject: [PATCH] simplify if clause

---
 kwant/solvers/mumps.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kwant/solvers/mumps.py b/kwant/solvers/mumps.py
index d672c4a1..f0b9ac0c 100644
--- a/kwant/solvers/mumps.py
+++ b/kwant/solvers/mumps.py
@@ -84,14 +84,14 @@ class Solver(common.SparseSolver):
             self.nrhs = nrhs
 
         if ordering is not None:
-            if ordering not in list(mumps.orderings.keys()) + ['kwant_decides']:
-                raise ValueError("Invalid ordering: " + ordering)
             if ordering == 'kwant_decides':
                 # Choose what is considered to be the best ordering.
                 sorted_orderings = [order
                                     for order in ['metis', 'scotch', 'auto']
                                     if order in mumps.possible_orderings()]
                 ordering = sorted_orderings[0]
+            elif ordering not in mumps.orderings:
+                raise ValueError("Invalid ordering: " + ordering)
             self.ordering = ordering
 
         if sparse_rhs is not None:
-- 
GitLab